[clang] clang/AMDGPU: Use all_equal instead of building a temporary set (PR #196742)
Matt Arsenault via cfe-commits
cfe-commits at lists.llvm.org
Sat May 9 11:13:21 PDT 2026
https://github.com/arsenm created https://github.com/llvm/llvm-project/pull/196742
Addresses comment on #196373
>From 29a9658f103f5983623f37e3bb8229af709640b0 Mon Sep 17 00:00:00 2001
From: Matt Arsenault <Matthew.Arsenault at amd.com>
Date: Sat, 9 May 2026 18:40:10 +0100
Subject: [PATCH] clang/AMDGPU: Use all_equal instead of building a temporary
set
Addresses comment on #196373
---
clang/lib/Driver/ToolChains/AMDGPU.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/clang/lib/Driver/ToolChains/AMDGPU.cpp b/clang/lib/Driver/ToolChains/AMDGPU.cpp
index b457ee2cde1c3..d2d8e56eb22be 100644
--- a/clang/lib/Driver/ToolChains/AMDGPU.cpp
+++ b/clang/lib/Driver/ToolChains/AMDGPU.cpp
@@ -739,8 +739,7 @@ AMDGPUToolChain::TranslateArgs(const DerivedArgList &Args, StringRef BoundArch,
<< getArchName() << llvm::toString(GPUsOrErr.takeError()) << "-mcpu";
} else {
auto &GPUs = *GPUsOrErr;
- if (llvm::SmallSet<llvm::StringRef, 1>(GPUs.begin(), GPUs.end()).size() >
- 1)
+ if (!llvm::all_equal(GPUs))
getDriver().Diag(diag::warn_drv_multi_gpu_arch)
<< getArchName() << llvm::join(GPUs, ", ") << "-mcpu";
DAL->AddJoinedArg(nullptr, Opts.getOption(options::OPT_mcpu_EQ),
More information about the cfe-commits
mailing list