================
@@ -622,7 +679,8 @@ bool SemaARM::CheckSMEBuiltinFunctionCall(unsigned 
BuiltinID,
     }
 
     if (BuiltinType)
-      checkArmStreamingBuiltin(SemaRef, TheCall, FD, *BuiltinType);
+      HasError |= checkArmStreamingBuiltin(SemaRef, TheCall, FD, *BuiltinType,
----------------
paulwalker-arm wrote:

Would it be wrong to return immediately? I ask because there's
```
  switch (BuiltinID) {
  default:
    return false;
```
which should be `return HasError;`? but if we can return directly then there's 
less change of other similar issues.

https://github.com/llvm/llvm-project/pull/93802
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to