aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFangrui Song <i@maskray.me>2024-03-27 13:50:17 -0700
committerFangrui Song <i@maskray.me>2024-03-27 13:50:17 -0700
commitf18600c87404eab8d0a279b0286f8add8b4a1bb8 (patch)
treeb0fdce2e6ea70e7cdcc7e2072e1149a453db1e92
parent70e8cf0c31493071c50c8112c6e0c7903abf6ca6 (diff)
downloadllvm-f18600c87404eab8d0a279b0286f8add8b4a1bb8.zip
llvm-f18600c87404eab8d0a279b0286f8add8b4a1bb8.tar.gz
llvm-f18600c87404eab8d0a279b0286f8add8b4a1bb8.tar.bz2
[Driver] Avoid repeated ToolChain.getTriple() calls. NFC
-rw-r--r--clang/lib/Driver/ToolChains/CommonArgs.cpp15
1 files changed, 7 insertions, 8 deletions
diff --git a/clang/lib/Driver/ToolChains/CommonArgs.cpp b/clang/lib/Driver/ToolChains/CommonArgs.cpp
index 6b1fbba..ace4fb9 100644
--- a/clang/lib/Driver/ToolChains/CommonArgs.cpp
+++ b/clang/lib/Driver/ToolChains/CommonArgs.cpp
@@ -758,15 +758,15 @@ bool tools::isTLSDESCEnabled(const ToolChain &TC,
void tools::addLTOOptions(const ToolChain &ToolChain, const ArgList &Args,
ArgStringList &CmdArgs, const InputInfo &Output,
const InputInfo &Input, bool IsThinLTO) {
- const bool IsOSAIX = ToolChain.getTriple().isOSAIX();
- const bool IsAMDGCN = ToolChain.getTriple().isAMDGCN();
+ const llvm::Triple &Triple = ToolChain.getTriple();
+ const bool IsOSAIX = Triple.isOSAIX();
+ const bool IsAMDGCN = Triple.isAMDGCN();
const char *Linker = Args.MakeArgString(ToolChain.GetLinkerPath());
const Driver &D = ToolChain.getDriver();
const bool IsFatLTO = Args.hasArg(options::OPT_ffat_lto_objects);
const bool IsUnifiedLTO = Args.hasArg(options::OPT_funified_lto);
if (llvm::sys::path::filename(Linker) != "ld.lld" &&
- llvm::sys::path::stem(Linker) != "ld.lld" &&
- !ToolChain.getTriple().isOSOpenBSD()) {
+ llvm::sys::path::stem(Linker) != "ld.lld" && !Triple.isOSOpenBSD()) {
// Tell the linker to load the plugin. This has to come before
// AddLinkerInputs as gold requires -plugin and AIX ld requires -bplugin to
// come before any -plugin-opt/-bplugin_opt that -Wl might forward.
@@ -835,7 +835,7 @@ void tools::addLTOOptions(const ToolChain &ToolChain, const ArgList &Args,
// the plugin.
// Handle flags for selecting CPU variants.
- std::string CPU = getCPUName(D, Args, ToolChain.getTriple());
+ std::string CPU = getCPUName(D, Args, Triple);
if (!CPU.empty())
CmdArgs.push_back(
Args.MakeArgString(Twine(PluginOptPrefix) + ExtraDash + "mcpu=" + CPU));
@@ -966,10 +966,9 @@ void tools::addLTOOptions(const ToolChain &ToolChain, const ArgList &Args,
bool HasRoptr = Args.hasFlag(options::OPT_mxcoff_roptr,
options::OPT_mno_xcoff_roptr, false);
StringRef OptStr = HasRoptr ? "-mxcoff-roptr" : "-mno-xcoff-roptr";
-
if (!IsOSAIX)
D.Diag(diag::err_drv_unsupported_opt_for_target)
- << OptStr << ToolChain.getTriple().str();
+ << OptStr << Triple.str();
if (HasRoptr) {
// The data sections option is on by default on AIX. We only need to error
@@ -1032,7 +1031,7 @@ void tools::addLTOOptions(const ToolChain &ToolChain, const ArgList &Args,
}
if (Args.hasFlag(options::OPT_femulated_tls, options::OPT_fno_emulated_tls,
- ToolChain.getTriple().hasDefaultEmulatedTLS())) {
+ Triple.hasDefaultEmulatedTLS())) {
CmdArgs.push_back(
Args.MakeArgString(Twine(PluginOptPrefix) + "-emulated-tls"));
}