sammccall added a subscriber: ABataev.
sammccall added a comment.

This seems vaguely reasonable to me, but I don't really know what an OpenMP 
capture is.

@ABataev, @jdoerfert : Any thoughts or concerns here?
Some context: an expression that contains errors can exist after certain types 
of sema/parse errors and acts as a placeholder in the AST with unknown 
semantics, e.g. we often don't know what its type is.
It is modeled as dependent, but its value/type/validity depends on how an the 
user would correct the error rather than how a template is instantiated.
(This behavior is with the -recovery-ast flag, which will one day be default).

@hokein Callers of `isDependentContext()` might mike sense to audit for 
recovery-AST-safety as it gives fewer guarantees than it used to.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80200/new/

https://reviews.llvm.org/D80200



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to