[Mlir-commits] [llvm] [mlir] [LLVM] Precise error message for intrinsic signature verification (2/n) (PR #199217)

[Rahul Joshi]

================
@@ -3,7 +3,7 @@
 ; Use of intrinsic without mangling suffix and invalid signature should
 ; be rejected.
 
-; CHECK: error: intrinsic has incorrect argument type!
+; CHECK: error: intrinsic argument 1 type (MatchType<0>) expected i8, but got i16
----------------
jurahul wrote:

For future changes, I was thinking of repeating the tablegen syntax (without the LLVM suffix) for dependent types in the error message, so would like to keep that consistent here. For example, for error messages related to LLVMExtendType, I was planning to use the `ExtendType<n>` in the error message. With that scheme, maybe better to keep using the MatchType<> syntax here? WDYT?

https://github.com/llvm/llvm-project/pull/199217