<div dir="ltr">Done in r173464.<div><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 24, 2013 at 6:05 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Can you craft a test case at least so we don't regress?</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jan 24, 2013 at 5:50 PM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: ddunbar<br>
Date: Thu Jan 24 19:50:34 2013<br>
New Revision: 173410<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=173410&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=173410&view=rev</a><br>
Log:<br>
[Frontend] The -iwithprefix option belongs in the After category, according to<br>
GCC docs.<br>
 - Found by inspection.<br>
<br>
Modified:<br>
    cfe/trunk/lib/Frontend/CompilerInvocation.cpp<br>
<br>
Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=173410&r1=173409&r2=173410&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=173410&r1=173409&r2=173410&view=diff</a><br>


==============================================================================<br>
--- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original)<br>
+++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Thu Jan 24 19:50:34 2013<br>
@@ -835,7 +835,7 @@<br>
     IsIndexHeaderMap = false;<br>
   }<br>
<br>
-  // Add -iprefix/-iwith-prefix/-iwithprefixbefore options.<br>
+  // Add -iprefix/-iwithprefix/-iwithprefixbefore options.<br>
   StringRef Prefix = ""; // FIXME: This isn't the correct default prefix.<br>
   for (arg_iterator it = Args.filtered_begin(OPT_iprefix, OPT_iwithprefix,<br>
                                              OPT_iwithprefixbefore),<br>
@@ -845,7 +845,7 @@<br>
       Prefix = A->getValue();<br>
     else if (A->getOption().matches(OPT_iwithprefix))<br>
       Opts.AddPath(Prefix.str() + A->getValue(),<br>
-                   frontend::System, false, false, false);<br>
+                   frontend::After, false, false, false);<br>
     else<br>
       Opts.AddPath(Prefix.str() + A->getValue(),<br>
                    frontend::Angled, false, false, false);<br>
<br>
<br>
_______________________________________________<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></div></blockquote></div><br></div>