[clang] clang/AMDGPU: Remove StringSwitch over r600 aliases (PR #209256)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 13 10:53:11 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Matt Arsenault (arsenm)
<details>
<summary>Changes</summary>
This will route to getArchNameR600 which should already handle
all the aliases.
---
Full diff: https://github.com/llvm/llvm-project/pull/209256.diff
1 Files Affected:
- (modified) clang/lib/Driver/ToolChains/CommonArgs.cpp (+4-13)
``````````diff
diff --git a/clang/lib/Driver/ToolChains/CommonArgs.cpp b/clang/lib/Driver/ToolChains/CommonArgs.cpp
index 2bfa1fbb1c678..08c06951cf220 100644
--- a/clang/lib/Driver/ToolChains/CommonArgs.cpp
+++ b/clang/lib/Driver/ToolChains/CommonArgs.cpp
@@ -697,19 +697,10 @@ void tools::AddTargetFeature(const ArgList &Args,
}
/// Get the (LLVM) name of the AMDGPU gpu we are targeting.
-static std::string getAMDGPUTargetGPU(const llvm::Triple &T,
- const ArgList &Args) {
+static StringRef getAMDGPUTargetGPU(const llvm::Triple &T,
+ const ArgList &Args) {
if (Arg *A = Args.getLastArg(options::OPT_mcpu_EQ)) {
- auto GPUName = getProcessorFromTargetID(T, A->getValue());
- return llvm::StringSwitch<std::string>(GPUName)
- .Cases({"rv630", "rv635"}, "r600")
- .Cases({"rv610", "rv620", "rs780"}, "rs880")
- .Case("rv740", "rv770")
- .Case("palm", "cedar")
- .Cases({"sumo", "sumo2"}, "sumo")
- .Case("hemlock", "cypress")
- .Case("aruba", "cayman")
- .Default(GPUName.str());
+ return getProcessorFromTargetID(T, A->getValue());
}
return "";
}
@@ -835,7 +826,7 @@ std::string tools::getCPUName(const Driver &D, const ArgList &Args,
case llvm::Triple::amdgpu:
case llvm::Triple::r600:
- return getAMDGPUTargetGPU(T, Args);
+ return getAMDGPUTargetGPU(T, Args).str();
case llvm::Triple::wasm32:
case llvm::Triple::wasm64:
``````````
</details>
https://github.com/llvm/llvm-project/pull/209256
More information about the cfe-commits
mailing list