================
@@ -7941,6 +7942,32 @@ static bool handleFunctionTypeAttr(TypeProcessingState 
&state, ParsedAttr &attr,
     return true;
   }
 
+  if (attr.getKind() == ParsedAttr::AT_CFISalt) {
+    if (attr.getNumArgs() == 0)
----------------
bwendling wrote:

> Perhaps? Alternatively, should this be an `assert(attr.getNumArgs() == 1)` ?

I don't want to do that in Sema. The check will go away in a non-Assert-Release 
build. And the compiler shouldn't ICE just because someone accidentally added 
an extra argument.

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

Reply via email to