<div dir="ltr">On Mon, Sep 30, 2013 at 12:46 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: chandlerc<br>
Date: Mon Sep 30 02:46:08 2013<br>
New Revision: 191653<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=191653&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=191653&view=rev</a><br>
Log:<br>
Add a bunch of other GCC '-f' options and ignore them. This covers every<br>
such option I've been able to find in the wild in our build system.<br>
<br>
Note that I haven't added test cases for this. I can do so if folks<br>
want, but they're *really* boring considering that this is all generated<br>
code to parse and ignore unsued options.<br></blockquote><div><br></div><div>I don't think these are quite right:</div><div><br></div><div>-ffunction-attribute-list, -finline-limit, -ftls-model, and -ftree-vectorizer-verbose need an "=argument" in GCC, and are rejected without. This patch seems to reject them with the argument but accept them without.</div>
<div><br></div><div>-fprofile-generate-sampling and -fprofile-reusedist are only accepted by the google branch of GCC, as far as I can see. Maybe that's fine, though -- I understand various groups use compilers built from that branch.</div>
<div><br></div><div>I can't find any GCC (neither trunk nor google branch) that accepts -fgnu nor -fprintf. Maybe these are typos in the relevant build rules?</div><div><br></div><div>-fspec-constr-count and -fregs-graph are GHC options, not GCC options.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Modified:<br>
    cfe/trunk/include/clang/Driver/Options.td<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=191653&r1=191652&r2=191653&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=191653&r1=191652&r2=191653&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/include/clang/Driver/Options.td (original)<br>
+++ cfe/trunk/include/clang/Driver/Options.td Mon Sep 30 02:46:08 2013<br>
@@ -1381,20 +1381,37 @@</blockquote><div>[...] </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-defm sse : BooleanFFlag<"see">, Group<clang_ignored_f_Group>;</blockquote>
<div><br></div><div>GCC does actually accept "-fsee", and ignores it. (This defm had a typo in its name, though.)</div></div></div></div>