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.
Please add a testcase! > 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
