================
@@ -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

Reply via email to