[TargetRegistry] Deprecate string overload of lookupTarget

[boomanaiden154]

Finish the TODO that was here in line with several of the other functions in TargetRegistry now that we have access to a triple through llvm::Module.

[boomanaiden154]

This is waiting on the following per-project cleanup PRs:

1. [NFC][clang] Prefer triple overload of lookupTarget #162185
2. [NFC][Flang] Prefer triple overloads of lookupTarget #162186
3. [NFC][MLIR] Prefer triple overload of lookupTarget #162187

[llvmbot]

@llvm/pr-subscribers-llvm-mc

Author: Aiden Grossman (boomanaiden154)

Changes

Finish the TODO that was here in line with several of the other functions in TargetRegistry now that we have access to a triple through llvm::Module.

---

Full diff: https://github.com/llvm/llvm-project/pull/162188.diff

1 Files Affected:

• (modified) llvm/include/llvm/MC/TargetRegistry.h (+2-1)

diff --git a/llvm/include/llvm/MC/TargetRegistry.h b/llvm/include/llvm/MC/TargetRegistry.h
index 570d4c0e8d272..234c587c8e951 100644
--- a/llvm/include/llvm/MC/TargetRegistry.h
+++ b/llvm/include/llvm/MC/TargetRegistry.h
@@ -737,7 +737,8 @@ struct TargetRegistry {
   /// \param TripleStr - The triple to use for finding a target.
   /// \param Error - On failure, an error string describing why no target was
   /// found.
-  // TODO: Drop this in favor of the method accepting Triple.
+  // TODO(boomanaiden154): Remove this function after LLVM 22 branches.
+  [[deprecated("Use overload accepting Triple instead")]]
   static const Target *lookupTarget(StringRef TripleStr, std::string &Error) {
     return lookupTarget(Triple(TripleStr), Error);
   }