================
@@ -272,9 +272,16 @@ class Intrinsic {
           Proto[I] == 'R' || Proto[I] == '@' || Proto[I] == '!')
         break;
 
+      if (Proto[I] == '2')
+        Param += 1;
+      if (Proto[I] == '4')
+        Param += 3;
----------------
kmclaughlin-arm wrote:

I see, it does make sense to try and avoid running into a subtle error in the 
future.
How about adding an assert for `Proto != '4'`? That way we're not adding any 
changes we can't yet test, but we can hopefully make it obvious that this needs 
to be updated in the future if this does need to support other types?

https://github.com/llvm/llvm-project/pull/165545
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to