================
@@ -1510,7 +1510,7 @@ static void CheckFoldOperand(Sema &S, Expr *E) {
E = E->IgnoreImpCasts();
auto *OCE = dyn_cast<CXXOperatorCallExpr>(E);
if ((OCE && OCE->isInfixBinaryOp()) || isa<BinaryOperator>(E) ||
- isa<AbstractConditionalOperator>(E)) {
+ isa<AbstractConditionalOperator>(E) || isa<RecoveryExpr>(E)) {
----------------
azmat-y wrote:
Yeah it no longer looks like a good idea to me either.
Instead of making any changes here instead maybe we can change
`Sema::ActOnConditionalOp`. After checking for absence of `LHSExpr` we can
check if our `CondExpr` contains Unexpanded parameter pack. If it does we don't
lower otherwise we do.
Thoughts?
https://github.com/llvm/llvm-project/pull/164019
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
