[cfe-commits] r173410 - /cfe/trunk/lib/Frontend/CompilerInvocation.cpp

[Richard Smith]

On Thu, Jan 24, 2013 at 5:50 PM, Daniel Dunbar <daniel at zuster.org> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits