[PATCH] D119375: [Clang][Sema] Do not evaluate value-dependent immediate invocations
Evgeny Shulgin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 9 13:48:48 PST 2022
Izaron added a comment.
The assert that I mentioned in the summary: https://github.com/llvm/llvm-project/blob/main/clang/lib/AST/ExprConstant.cpp#L14969
(Although `assert`s are deleted in release builds, one can check the violation by `llvm::errs() << isValueDependent() << "\n";` in the method)
I think I have found the right place to check the condition, right after creating the `ConstantExpr` object.
//P.S. If this review is eventually approved, kindly please merge the commit on my behalf =) As I don't have merge access. My name is `Evgeny Shulgin` and email is `izaronplatz at gmail.com`. Sorry for inconvenience!//
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits