<div dir="ltr">Thanks for the review.<div><br></div><div>Sorry, I didn't checkout the latest file. It has been already fixed in <a href="https://github.com/llvm-mirror/clang/commit/4aaedf03f5828c6fec07b28e723871ec7ef3ff5a">https://github.com/llvm-mirror/clang/commit/4aaedf03f5828c6fec07b28e723871ec7ef3ff5a</a> .</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 October 2013 19:48, Anna Zaks <span dir="ltr"><<a href="mailto:ganna@apple.com" target="_blank">ganna@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Adding these should come after checking IgnoredOptionMap since some options need to be ignored.<div><div>  # Ignored options.</div><div>  if (defined $IgnoredOptionMap{$ArgKey}) {</div><div>
    my $Cnt = $IgnoredOptionMap{$ArgKey};</div><div>    while ($Cnt > 0) {</div><div>      ++$i; --$Cnt;</div><div>    }</div><div>    next;</div><div>  }</div><div><br></div><div>Also, I would have thought that the following would add the valid -f options to the command line. Why that does not work?</div>
<div><br></div><div><div>  if ($Arg =~ /^-f/) {</div><div>    push @CompileOpts,$Arg;</div><div>    push @LinkOpts,$Arg;</div><div>    next;</div><div>  }</div></div><div><br></div><div>Thanks,</div><div>Anna.</div><div><div>
<div class="h5"><div>On Oct 1, 2013, at 11:32 PM, Liang Qi <<a href="mailto:cavendish.qi@gmail.com" target="_blank">cavendish.qi@gmail.com</a>> wrote:</div><br></div></div><blockquote type="cite"><div><div class="h5">
<div dir="ltr"><div>Hi,</div><div><br></div><div>This change whitelists -fXXXX options. At least the use case here is to pass -fPIE and -fPIC to compiler, more talks in <a href="https://bugreports.qt-project.org/browse/QTBUG-33698" target="_blank">https://bugreports.qt-project.org/browse/QTBUG-33698</a> .</div>

<div><br></div><div>Review appreciated.</div><div><br></div><div>Regards,</div><div>Liang</div><div><br></div><div><div style="font-size:13.333333969116211px;margin-bottom:0px;margin-left:0px;padding-bottom:5px;font-family:arial,sans-serif">

<div style="overflow:hidden"><a href="http://llvm-reviews.chandlerc.com/D1805" target="_blank">http://llvm-reviews.chandlerc.com/D1805</a><br><br>Files:<br>  tools/scan-build/ccc-analyzer<br><br>Index: tools/scan-build/ccc-analyzer<br>

===================================================================<br>--- tools/scan-build/ccc-analyzer<br>+++ tools/scan-build/ccc-analyzer<br>@@ -488,6 +488,10 @@<br>     push @CompileOpts,$Arg;<br>     next;<br>   }<br>

+  if ($Arg =~ /-f.*/) {<br>+    push @CompileOpts,$Arg;<br>+    next;<br>+  }<br>   # Handle the case where there isn't a space after -iquote<br>   if ($Arg =~ /-iquote.*/) {<br>     push @CompileOpts,$Arg;<div>
</div></div></div><div style="font-size:13.333333969116211px;font-family:arial,sans-serif"></div></div><div><br></div><br></div></div></div>
_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><a href="http://www.qiliang.net">http://www.qiliang.net</a>
</div>