tra added a comment.
Perhaps we should allow this in all unevaluated contexts?
I.e. `int s = sizeof(foo(x));` should also work.
================
Comment at: clang/include/clang/Sema/Sema.h:10411
+ auto I =
+ std::find_if(ExprEvalContexts.rbegin(), ExprEvalContexts.rend(),
+ [](const ExpressionEvaluationContextRecord &C) {
----------------
I think you want `return llvm::any_of(ExprEvalContexts, ...)` here and you can
fold it directly into `if()` below.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61458/new/
https://reviews.llvm.org/D61458
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
