[clang] clang: Remove copy of string into temporary set (PR #196373)
via cfe-commits
cfe-commits at lists.llvm.org
Thu May 7 10:16:46 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-driver
Author: Matt Arsenault (arsenm)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/196373.diff
1 Files Affected:
- (modified) clang/lib/Driver/ToolChains/AMDGPU.cpp (+2-1)
``````````diff
diff --git a/clang/lib/Driver/ToolChains/AMDGPU.cpp b/clang/lib/Driver/ToolChains/AMDGPU.cpp
index 3a1e82ec01b74..b457ee2cde1c3 100644
--- a/clang/lib/Driver/ToolChains/AMDGPU.cpp
+++ b/clang/lib/Driver/ToolChains/AMDGPU.cpp
@@ -739,7 +739,8 @@ AMDGPUToolChain::TranslateArgs(const DerivedArgList &Args, StringRef BoundArch,
<< getArchName() << llvm::toString(GPUsOrErr.takeError()) << "-mcpu";
} else {
auto &GPUs = *GPUsOrErr;
- if (llvm::SmallSet<std::string, 1>(GPUs.begin(), GPUs.end()).size() > 1)
+ if (llvm::SmallSet<llvm::StringRef, 1>(GPUs.begin(), GPUs.end()).size() >
+ 1)
getDriver().Diag(diag::warn_drv_multi_gpu_arch)
<< getArchName() << llvm::join(GPUs, ", ") << "-mcpu";
DAL->AddJoinedArg(nullptr, Opts.getOption(options::OPT_mcpu_EQ),
``````````
</details>
https://github.com/llvm/llvm-project/pull/196373
More information about the cfe-commits
mailing list