"Chad Rosier" <[email protected]> writes: > A test case might not be a bad idea..
Can't get away with anything around here ;) Added a test for the new flag and a bonus test for the previously untested -fprofile-arcs in r215053. >> Author: bogner >> Date: Wed Aug 6 22:14:34 2014 >> New Revision: 215051 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=215051&view=rev> Log: >> Driver: Add -fno-profile-arcs to go with -fprofile-arcs >> >> This is a trivial gcc-compatible change. >> >> Modified: >> cfe/trunk/include/clang/Driver/Options.td >> cfe/trunk/lib/Driver/ToolChains.cpp >> cfe/trunk/lib/Driver/Tools.cpp >> >> Modified: cfe/trunk/include/clang/Driver/Options.td >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=215051&r1=215050&r2=215051&view=diff> >> >> ============================================================================== >> --- cfe/trunk/include/clang/Driver/Options.td (original) >> +++ cfe/trunk/include/clang/Driver/Options.td Wed Aug 6 22:14:34 2014 >> @@ -822,6 +822,7 @@ def fno_pic : Flag<["-"], "fno-pic">, Gr >> def fpie : Flag<["-"], "fpie">, Group<f_Group>; >> def fno_pie : Flag<["-"], "fno-pie">, Group<f_Group>; >> def fprofile_arcs : Flag<["-"], "fprofile-arcs">, Group<f_Group>; >> +def fno_profile_arcs : Flag<["-"], "fno-profile-arcs">, Group<f_Group>; >> def fprofile_generate : Flag<["-"], "fprofile-generate">, Group<f_Group>; >> def framework : Separate<["-"], "framework">, Flags<[LinkerInput]>; >> def frandom_seed_EQ : Joined<["-"], "frandom-seed=">, >> Group<clang_ignored_f_Group>; >> >> Modified: cfe/trunk/lib/Driver/ToolChains.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=215051&r1=215050&r2=215051&view=diff> >> >> ============================================================================== >> --- cfe/trunk/lib/Driver/ToolChains.cpp (original) >> +++ cfe/trunk/lib/Driver/ToolChains.cpp Wed Aug 6 22:14:34 2014 >> @@ -333,7 +333,8 @@ void DarwinClang::AddLinkRuntimeLibArgs( >> } >> >> // If we are building profile support, link that library in. >> - if (Args.hasArg(options::OPT_fprofile_arcs) || >> + if (Args.hasFlag(options::OPT_fprofile_arcs, >> options::OPT_fno_profile_arcs, >> + false) || >> Args.hasArg(options::OPT_fprofile_generate) || >> Args.hasArg(options::OPT_fprofile_instr_generate) || >> Args.hasArg(options::OPT_fcreate_profile) || >> >> Modified: cfe/trunk/lib/Driver/Tools.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=215051&r1=215050&r2=215051&view=diff> >> >> ============================================================================== >> --- cfe/trunk/lib/Driver/Tools.cpp (original) >> +++ cfe/trunk/lib/Driver/Tools.cpp Wed Aug 6 22:14:34 2014 >> @@ -2105,7 +2105,8 @@ static void addClangRTLinux( >> >> static void addProfileRT( >> const ToolChain &TC, const ArgList &Args, ArgStringList &CmdArgs) { >> - if (!(Args.hasArg(options::OPT_fprofile_arcs) || >> + if (!(Args.hasFlag(options::OPT_fprofile_arcs, >> options::OPT_fno_profile_arcs, >> + false) || >> Args.hasArg(options::OPT_fprofile_generate) || >> Args.hasArg(options::OPT_fprofile_instr_generate) || >> Args.hasArg(options::OPT_fcreate_profile) || >> @@ -3226,7 +3227,8 @@ void Clang::ConstructJob(Compilation &C, >> if (Args.hasArg(options::OPT_ftest_coverage) || >> Args.hasArg(options::OPT_coverage)) >> CmdArgs.push_back("-femit-coverage-notes"); >> - if (Args.hasArg(options::OPT_fprofile_arcs) || >> + if (Args.hasFlag(options::OPT_fprofile_arcs, >> options::OPT_fno_profile_arcs, >> + false) || >> Args.hasArg(options::OPT_coverage)) >> CmdArgs.push_back("-femit-coverage-data"); >> >> >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits> _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
