maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Driver] Don't pass default value to getCompilerRTArgString 

Using static library is already a default.

Differential Revision: https://reviews.llvm.org/D56043

llvm-svn: 351710
This commit is contained in:
Petr Hosek 2019-01-21 01:34:09 +00:00
parent 6d58c9e2b2
commit 9c178356e0
4 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
clang/lib/Driver/ToolChains/CommonArgs.cpp
View File

@ -757,9 +757,9 @@ bool tools::addXRayRuntime(const ToolChain&TC, const ArgList &Args, ArgStringLis
if (TC.getXRayArgs().needsXRayRt()) { if (TC.getXRayArgs().needsXRayRt()) {
CmdArgs.push_back("-whole-archive"); CmdArgs.push_back("-whole-archive");
CmdArgs.push_back(TC.getCompilerRTArgString(Args, "xray", false)); CmdArgs.push_back(TC.getCompilerRTArgString(Args, "xray"));
for (const auto &Mode : TC.getXRayArgs().modeList()) for (const auto &Mode : TC.getXRayArgs().modeList())
CmdArgs.push_back(TC.getCompilerRTArgString(Args, Mode, false)); CmdArgs.push_back(TC.getCompilerRTArgString(Args, Mode));
CmdArgs.push_back("-no-whole-archive"); CmdArgs.push_back("-no-whole-archive");
return true; return true;
} }

2
clang/lib/Driver/ToolChains/MSVC.cpp
View File

@ -377,7 +377,7 @@ void visualstudio::Linker::ConstructJob(Compilation &C, const JobAction &JA,
if (!Args.hasArg(options::OPT_shared)) if (!Args.hasArg(options::OPT_shared))
CmdArgs.push_back( CmdArgs.push_back(
Args.MakeArgString(std::string("-wholearchive:") + Args.MakeArgString(std::string("-wholearchive:") +
TC.getCompilerRTArgString(Args, "fuzzer", false))); TC.getCompilerRTArgString(Args, "fuzzer")));
CmdArgs.push_back(Args.MakeArgString("-debug")); CmdArgs.push_back(Args.MakeArgString("-debug"));
// Prevent the linker from padding sections we use for instrumentation // Prevent the linker from padding sections we use for instrumentation
// arrays. // arrays.

4
clang/lib/Driver/ToolChains/OpenBSD.cpp
View File

@ -188,11 +188,11 @@ void openbsd::Linker::ConstructJob(Compilation &C, const JobAction &JA,
CmdArgs.push_back("-lm"); CmdArgs.push_back("-lm");
} }
if (NeedsSanitizerDeps) { if (NeedsSanitizerDeps) {
CmdArgs.push_back(ToolChain.getCompilerRTArgString(Args, "builtins", false)); CmdArgs.push_back(ToolChain.getCompilerRTArgString(Args, "builtins"));
linkSanitizerRuntimeDeps(ToolChain, CmdArgs); linkSanitizerRuntimeDeps(ToolChain, CmdArgs);
} }
if (NeedsXRayDeps) { if (NeedsXRayDeps) {
CmdArgs.push_back(ToolChain.getCompilerRTArgString(Args, "builtins", false)); CmdArgs.push_back(ToolChain.getCompilerRTArgString(Args, "builtins"));
linkXRayRuntimeDeps(ToolChain, CmdArgs); linkXRayRuntimeDeps(ToolChain, CmdArgs);
} }
// FIXME: For some reason GCC passes -lgcc before adding // FIXME: For some reason GCC passes -lgcc before adding

4
clang/lib/Driver/ToolChains/Solaris.cpp
View File

@ -100,7 +100,7 @@ void solaris::Linker::ConstructJob(Compilation &C, const JobAction &JA,
// __start_SECNAME labels. // __start_SECNAME labels.
CmdArgs.push_back("--whole-archive"); CmdArgs.push_back("--whole-archive");
CmdArgs.push_back( CmdArgs.push_back(
getToolChain().getCompilerRTArgString(Args, "sancov_begin", false)); getToolChain().getCompilerRTArgString(Args, "sancov_begin"));
CmdArgs.push_back("--no-whole-archive"); CmdArgs.push_back("--no-whole-archive");
getToolChain().AddFilePathLibArgs(Args, CmdArgs); getToolChain().AddFilePathLibArgs(Args, CmdArgs);
@ -135,7 +135,7 @@ void solaris::Linker::ConstructJob(Compilation &C, const JobAction &JA,
// __stop_SECNAME labels. // __stop_SECNAME labels.
CmdArgs.push_back("--whole-archive"); CmdArgs.push_back("--whole-archive");
CmdArgs.push_back( CmdArgs.push_back(
getToolChain().getCompilerRTArgString(Args, "sancov_end", false)); getToolChain().getCompilerRTArgString(Args, "sancov_end"));
CmdArgs.push_back("--no-whole-archive"); CmdArgs.push_back("--no-whole-archive");
if (!Args.hasArg(options::OPT_nostdlib, options::OPT_nostartfiles)) { if (!Args.hasArg(options::OPT_nostdlib, options::OPT_nostartfiles)) {