[llvm] [LLVM][TableGen] Add overloaded intrinsic name conflict checks (PR #109314)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 10:16:02 PDT 2024


================
@@ -94,6 +95,168 @@ void CodeGenIntrinsicTable::CheckDuplicateIntrinsics() const {
   PrintFatalNote(First.TheDef, "Previous definition here");
 }
 
+// Note: This is a modified version of `Intrinsic::lookupLLVMIntrinsicByName`
+// in IntrinsicInst.cpp file.
+template <typename T>
----------------
jurahul wrote:

1st point of feedback: I made this templated so that we can share the same code here and in `Intrinsic::lookupLLVMIntrinsicByName`. I am proposing we move this to a common header in Support that both pieces of code use.

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


More information about the llvm-commits mailing list