<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Wed, Jun 20, 2012 at 12:03 AM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com" target="_blank">kremenek@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: kremenek<br>
Date: Wed Jun 20 02:03:37 2012<br>
New Revision: 158796<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=158796&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=158796&view=rev</a><br>
Log:<br>
Revert "Provide a -no-pedantic to cancel out -pedantic."  This needs to be designed<br>
a bit further.  We may wish to just have -Wno flags to silence warnings, and not have a -no-pedantic.<br></blockquote><div><br></div><div>Why? This seems to make using '-pedantic' fairly problematic...</div><div>
<br></div><div>A lot of build systems essentially only allow appending flags to change the build flags in use, and it seems bad to leave flags around which "poison" the entire command because of a lack of a '-no-*' variant.</div>
<div><br></div><div>Note that I'm not really endorsing the use of '-pedantic'; I generally prefer the -W flags (with their -Wno- variants). I'm just a bit concerned about not allowing appending an option to reverse the decision about '-pedantic'.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Modified:<br>
    cfe/trunk/include/clang/Driver/Options.td<br>
    cfe/trunk/lib/Driver/Tools.cpp<br>
    cfe/trunk/test/Driver/warning-options.cpp<br>
<br>
Modified: cfe/trunk/include/clang/Driver/Options.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=158796&r1=158795&r2=158796&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=158796&r1=158795&r2=158796&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/include/clang/Driver/Options.td (original)<br>
+++ cfe/trunk/include/clang/Driver/Options.td Wed Jun 20 02:03:37 2012<br>
@@ -838,7 +838,6 @@<br>
 def no_cpp_precomp : Flag<"-no-cpp-precomp">, Group<clang_ignored_f_Group>;<br>
 def no_integrated_as : Flag<"-no-integrated-as">, Flags<[DriverOption]>;<br>
 def no_integrated_cpp : Flag<"-no-integrated-cpp">, Flags<[DriverOption]>;<br>
-def no_pedantic : Flag<"-no-pedantic">, Group<pedantic_Group>;<br>
 def no__dead__strip__inits__and__terms : Flag<"-no_dead_strip_inits_and_terms">;<br>
 def nobuiltininc : Flag<"-nobuiltininc">, Flags<[CC1Option]>,<br>
   HelpText<"Disable builtin #include directories">;<br>
@@ -1041,7 +1040,6 @@<br>
 def _machine : Separate<"--machine">, Alias<m_Joined>;<br>
 def _no_integrated_cpp : Flag<"--no-integrated-cpp">, Alias<no_integrated_cpp>;<br>
 def _no_line_commands : Flag<"--no-line-commands">, Alias<P>;<br>
-def _no_pedantic : Flag<"--no-pedantic">, Alias<no_pedantic>;<br>
 def _no_standard_includes : Flag<"--no-standard-includes">, Alias<nostdinc>;<br>
 def _no_standard_libraries : Flag<"--no-standard-libraries">, Alias<nostdlib>;<br>
 def _no_undefined : Flag<"--no-undefined">, Flags<[LinkerInput]>;<br>
<br>
Modified: cfe/trunk/lib/Driver/Tools.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=158796&r1=158795&r2=158796&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=158796&r1=158795&r2=158796&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/lib/Driver/Tools.cpp (original)<br>
+++ cfe/trunk/lib/Driver/Tools.cpp Wed Jun 20 02:03:37 2012<br>
@@ -2034,8 +2034,7 @@<br>
   }<br>
<br>
   Args.AddAllArgs(CmdArgs, options::OPT_W_Group);<br>
-  if (Args.hasFlag(options::OPT_pedantic, options::OPT_no_pedantic, false))<br>
-    CmdArgs.push_back("-pedantic");<br>
+  Args.AddLastArg(CmdArgs, options::OPT_pedantic);<br>
   Args.AddLastArg(CmdArgs, options::OPT_pedantic_errors);<br>
   Args.AddLastArg(CmdArgs, options::OPT_w);<br>
<br>
<br>
Modified: cfe/trunk/test/Driver/warning-options.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=158796&r1=158795&r2=158796&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=158796&r1=158795&r2=158796&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/test/Driver/warning-options.cpp (original)<br>
+++ cfe/trunk/test/Driver/warning-options.cpp Wed Jun 20 02:03:37 2012<br>
@@ -9,10 +9,6 @@<br>
 // CHECK: unknown warning option '-Wno-monkey'<br>
 // CHECK: unknown warning option '-Wno-unused-command-line-arguments'; did you mean '-Wno-unused-command-line-argument'?<br>
<br>
-// RUN: %clang -### -pedantic -no-pedantic %s 2>&1 | FileCheck -check-prefix=NO_PEDANTIC %s<br>
-// NO_PEDANTIC-NOT: -pedantic<br>
-// RUN: %clang -### -pedantic -pedantic -no-pedantic -pedantic %s 2>&1 | FileCheck -check-prefix=PEDANTIC %s<br>
-// PEDANTIC: -pedantic<br>
-<br>
 // PR12920: They use gcc driver. Clang tends to pass -pedantic to gcc-as and gcc-ld.<br>
 // XFAIL: cygwin,mingw32<br>
+<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></font></div>