[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