Done in r173464. - Daniel
On Thu, Jan 24, 2013 at 6:05 PM, Chandler Carruth <[email protected]>wrote: > Can you craft a test case at least so we don't regress? > > > On Thu, Jan 24, 2013 at 5:50 PM, Daniel Dunbar <[email protected]> wrote: > >> Author: ddunbar >> Date: Thu Jan 24 19:50:34 2013 >> New Revision: 173410 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=173410&view=rev >> Log: >> [Frontend] The -iwithprefix option belongs in the After category, >> according to >> GCC docs. >> - Found by inspection. >> >> Modified: >> cfe/trunk/lib/Frontend/CompilerInvocation.cpp >> >> Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=173410&r1=173409&r2=173410&view=diff >> >> ============================================================================== >> --- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original) >> +++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Thu Jan 24 19:50:34 2013 >> @@ -835,7 +835,7 @@ >> IsIndexHeaderMap = false; >> } >> >> - // Add -iprefix/-iwith-prefix/-iwithprefixbefore options. >> + // Add -iprefix/-iwithprefix/-iwithprefixbefore options. >> StringRef Prefix = ""; // FIXME: This isn't the correct default prefix. >> for (arg_iterator it = Args.filtered_begin(OPT_iprefix, >> OPT_iwithprefix, >> OPT_iwithprefixbefore), >> @@ -845,7 +845,7 @@ >> Prefix = A->getValue(); >> else if (A->getOption().matches(OPT_iwithprefix)) >> Opts.AddPath(Prefix.str() + A->getValue(), >> - frontend::System, false, false, false); >> + frontend::After, false, false, false); >> else >> Opts.AddPath(Prefix.str() + A->getValue(), >> frontend::Angled, false, false, false); >> >> >> _______________________________________________ >> 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
