[clang] [clang] fix sema init crash for not checking a ExprResult (PR #98102)

Davide Italiano via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 9 07:56:55 PDT 2024


================
@@ -5576,6 +5576,10 @@ static void TryOrBuildParenListInitialization(
       ExprResult ER;
       ER = IS.Perform(S, SubEntity, SubKind,
                       Arg ? MultiExprArg(Arg) : std::nullopt);
+
+      if (ER.isInvalid())
----------------
dcci wrote:

@AaronBallman -- are you OK with a crash-on-invalid (the one in first post)? The original example we had crashed on valid code, but reducing a 35MB file with a lot of SFINAE is taking weeks with c-reduce.
Thanks for your review :)

https://github.com/llvm/llvm-project/pull/98102


More information about the cfe-commits mailing list