From: Hans Wennborg Date: Tue, 17 Sep 2013 00:03:41 +0000 (+0000) Subject: clang-cl: ignore compile-only options in link-only invocations. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=d8df3f70930b2251119ed363a7fde737c76af784;p=clang clang-cl: ignore compile-only options in link-only invocations. Previously we would warn about unused arguments such as /MD when linking. Clang already has logic to ignore compile-only options, e.g. for -D and -U. This patch extends that to include clang-cl's compile-only options too. Also, some clang-cl options should always be ignored. Doing this earlier means they get ignored both for compilation and link-only invocations. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@190825 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Driver/CLCompatOptions.td b/include/clang/Driver/CLCompatOptions.td index 1941c608c9..4dbb1ca5d0 100644 --- a/include/clang/Driver/CLCompatOptions.td +++ b/include/clang/Driver/CLCompatOptions.td @@ -14,28 +14,43 @@ def cl_Group : OptionGroup<"">, HelpText<"CL.EXE COMPATIBILITY OPTIONS">; +def cl_compile_Group : OptionGroup<"">, + Group; + def cl_ignored_Group : OptionGroup<"">, Group; class CLFlag : Option<["/", "-"], name, KIND_FLAG>, Group, Flags<[CLOption, DriverOption]>; +class CLCompileFlag : Option<["/", "-"], name, KIND_FLAG>, + Group, Flags<[CLOption, DriverOption]>; + class CLIgnoredFlag : Option<["/", "-"], name, KIND_FLAG>, Group, Flags<[CLOption, DriverOption]>; class CLJoined : Option<["/", "-"], name, KIND_JOINED>, Group, Flags<[CLOption, DriverOption]>; +class CLCompileJoined : Option<["/", "-"], name, KIND_JOINED>, + Group, Flags<[CLOption, DriverOption]>; + class CLIgnoredJoined : Option<["/", "-"], name, KIND_JOINED>, Group, Flags<[CLOption, DriverOption]>; class CLJoinedOrSeparate : Option<["/", "-"], name, KIND_JOINED_OR_SEPARATE>, Group, Flags<[CLOption, DriverOption]>; +class CLCompileJoinedOrSeparate : Option<["/", "-"], name, + KIND_JOINED_OR_SEPARATE>, Group, + Flags<[CLOption, DriverOption]>; + class CLRemainingArgs : Option<["/", "-"], name, KIND_REMAINING_ARGS>, Group, Flags<[CLOption, DriverOption]>; // Aliases: +// (We don't put any of these in cl_compile_Group as the options they alias are +// already in the right group.) def _SLASH_c : CLFlag<"c">, HelpText<"Compile only">, Alias; def _SLASH_D : CLJoinedOrSeparate<"D">, HelpText<"Define macro">, @@ -93,12 +108,12 @@ def _SLASH_Zs : CLFlag<"Zs">, HelpText<"Syntax-check only">, // Non-aliases: -def _SLASH_M_Group : OptionGroup<"">, Group; +def _SLASH_M_Group : OptionGroup<"">, Group; def _SLASH_Fe : CLJoined<"Fe">, HelpText<"Set output executable file or directory (ends in / or \\)">, MetaVarName<"">; -def _SLASH_Fo : CLJoined<"Fo">, +def _SLASH_Fo : CLCompileJoined<"Fo">, HelpText<"Set output object file, or directory (ends in / or \\)">, MetaVarName<"">; def _SLASH_LD : CLFlag<"LD">, HelpText<"Create DLL">; @@ -113,12 +128,12 @@ def _SLASH_MT : Option<["/", "-"], "MT", KIND_FLAG>, Group<_SLASH_M_Group>, Flags<[CLOption, DriverOption]>, HelpText<"Use static run-time">; def _SLASH_MTd : Option<["/", "-"], "MTd", KIND_FLAG>, Group<_SLASH_M_Group>, Flags<[CLOption, DriverOption]>, HelpText<"Use static debug run-time">; -def _SLASH_Tc : CLJoinedOrSeparate<"Tc">, HelpText<"Specify a C source file">, - MetaVarName<"">; -def _SLASH_TC : CLFlag<"TC">, HelpText<"Treat all source files as C">; -def _SLASH_Tp : CLJoinedOrSeparate<"Tp">, HelpText<"Specify a C++ source file">, - MetaVarName<"">; -def _SLASH_TP : CLFlag<"TP">, HelpText<"Treat all source files as C++">; +def _SLASH_Tc : CLCompileJoinedOrSeparate<"Tc">, + HelpText<"Specify a C source file">, MetaVarName<"">; +def _SLASH_TC : CLCompileFlag<"TC">, HelpText<"Treat all source files as C">; +def _SLASH_Tp : CLCompileJoinedOrSeparate<"Tp">, + HelpText<"Specify a C++ source file">, MetaVarName<"">; +def _SLASH_TP : CLCompileFlag<"TP">, HelpText<"Treat all source files as C++">; // Ignored: diff --git a/lib/Driver/Driver.cpp b/lib/Driver/Driver.cpp index b1d5e61963..7ad09accd8 100644 --- a/lib/Driver/Driver.cpp +++ b/lib/Driver/Driver.cpp @@ -1268,8 +1268,13 @@ void Driver::BuildActions(const ToolChain &TC, DerivedArgList &Args, // If we are linking, claim any options which are obviously only used for // compilation. - if (FinalPhase == phases::Link && PL.size() == 1) + if (FinalPhase == phases::Link && PL.size() == 1) { Args.ClaimAllArgs(options::OPT_CompileOnly_Group); + Args.ClaimAllArgs(options::OPT_cl_compile_Group); + } + + // Claim ignored clang-cl options. + Args.ClaimAllArgs(options::OPT_cl_ignored_Group); } Action *Driver::ConstructPhaseAction(const ArgList &Args, phases::ID Phase, diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp index 305cfb48d2..c1bac556cd 100644 --- a/lib/Driver/Tools.cpp +++ b/lib/Driver/Tools.cpp @@ -3587,9 +3587,6 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, Args.ClaimAllArgs(options::OPT_clang_ignored_f_Group); Args.ClaimAllArgs(options::OPT_clang_ignored_m_Group); - // Claim ignored clang-cl options. - Args.ClaimAllArgs(options::OPT_cl_ignored_Group); - // Disable warnings for clang -E -emit-llvm foo.c Args.ClaimAllArgs(options::OPT_emit_llvm); } diff --git a/test/Driver/cl-options.c b/test/Driver/cl-options.c index f74a16c433..f44c4169d0 100644 --- a/test/Driver/cl-options.c +++ b/test/Driver/cl-options.c @@ -93,11 +93,19 @@ // WJoined: "-cc1" // WJoined: "-Wunused-pragmas" + // Ignored options. Check that we don't get "unused during compilation" errors. // (/Zs is for syntax-only, /WX is for -Werror) // RUN: %clang_cl /Zs /WX /analyze- /errorReport:foo /nologo /Ob1 /Ob2 -- %s // RUN: %clang_cl /Zs /WX /Zc:forScope /Zc:wchar_t /w12345 /wd1234 /RTC1 -- %s +// Ignored options and compile-only options are ignored for link jobs. +// RUN: %clang_cl /c /Fo%t.obj /Tc%s +// RUN: %clang_cl /nologo -### -- %t.obj 2>&1 | FileCheck -check-prefix=LINKUNUSED %s +// RUN: %clang_cl /Dfoo -### -- %t.obj 2>&1 | FileCheck -check-prefix=LINKUNUSED %s +// RUN: %clang_cl /MD -### -- %t.obj 2>&1 | FileCheck -check-prefix=LINKUNUSED %s +// LINKUNUSED-NOT: argument unused during compilation + // Support ignoring warnings about unused arguments. // RUN: %clang_cl /Abracadabra -Qunused-arguments -### -- %s 2>&1 | FileCheck -check-prefix=UNUSED %s // UNUSED-NOT: warning @@ -116,3 +124,5 @@ // RUN: %clang_cl -Xclang hellocc1 -### -- %s 2>&1 | FileCheck -check-prefix=Xclang %s // Xclang: "-cc1" // Xclang: "hellocc1" + +void f() { }