[llvm] [Core] Skip over target name in intrinsic name lookup (PR #109971)
Rahul Joshi via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 25 05:35:05 PDT 2024
================
@@ -963,8 +963,9 @@ bool Function::isTargetIntrinsic() const {
/// Find the segment of \c IntrinsicNameTable for intrinsics with the same
/// target as \c Name, or the generic table if \c Name is not target specific.
///
-/// Returns the relevant slice of \c IntrinsicNameTable
-static ArrayRef<const char *> findTargetSubtable(StringRef Name) {
+/// Returns the relevant slice of \c IntrinsicNameTable and the target name.
+static std::pair<ArrayRef<const char *>, StringRef>
+findTargetSubtable(StringRef Name) {
----------------
jurahul wrote:
As a side note, this function seems to be only called from `lookupIntrinsicID` and is small enough to be inlined. May be we can do that as a separate NFC?
https://github.com/llvm/llvm-project/pull/109971
More information about the llvm-commits
mailing list