[llvm] [LLVM][Parser] Check invalid overload suffix for intrinsics (PR #108315)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 12 05:01:06 PDT 2024


jurahul wrote:

The problem I ran into was literally this: I was writing a unit test using overloaded intrinsics and observed that no one complained if I messed up the mangling suffix (use .f16 for a float variant for instance), and that motivated this change. 

I am actually not sure what happens if in C++ someone creates a Function::Create() to create an intrinsic decl but with bad overload suffix. I'd expect that to be strict, not sure if it is.

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


More information about the llvm-commits mailing list