================
@@ -10165,8 +10165,11 @@ static bool checkOpenMPIterationSpace(
     HasErrors |= ISC.checkAndSetInc(For ? For->getInc() : CXXFor->getInc());
   }
 
-  if (ISC.dependent() || SemaRef.CurContext->isDependentContext() || HasErrors)
+  if (SemaRef.CurContext->isDependentContext() || HasErrors)
     return HasErrors;
+  // Outside of a dependent context this only happens during error recovery.
----------------
ojhunt wrote:

This is not true -- I spent some time looking at the block test the LLM 
introduced in the last revision, blocks do not report a dependent context, 
which _might_ be a separate bug. Lambdas are semantically similar to blocks but 
label their operator() as being dependent plus a bunch of other weirdness.

While seeing if I could get problems with lambdas was able to induce a pile of 
lambda weirdness (https://github.com/llvm/llvm-project/issues/227596)


https://github.com/llvm/llvm-project/pull/226805
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to