================ @@ -5576,6 +5576,10 @@ static void TryOrBuildParenListInitialization( ExprResult ER; ER = IS.Perform(S, SubEntity, SubKind, Arg ? MultiExprArg(Arg) : std::nullopt); + + if (ER.isInvalid()) ---------------- erichkeane wrote:
For `ExprResult`, I tend to find `isUsable` to be more clear, and I thought is slightly more strict, as it also catches cases where the ER is unset. That said, we use `isInvalid` all over the place, assuming that it would have been set to SOMETHING. https://github.com/llvm/llvm-project/pull/98102 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits