patch: don't use -backend-option for vectorizer flags

Richard Smith richard at metafoo.co.uk
Mon Jun 24 18:45:01 PDT 2013


LGTM

On Mon, Jun 24, 2013 at 6:35 PM, Nick Lewycky <nlewycky at google.com> wrote:
> Currently -fvectorize and friends inject themselves into the BackendArgs so
> that they enable the command line flags when the command line parser runs in
> BackendUtil.
>
> This is incompatible with folks trying to use clang as a library to build
> more than one translation unit (as flags are global, and you may only
> specify -fvectorize once). Here's an example of the symptoms of the problem:
>   http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-June/030264.html
>
> The attached patch creates new CC1 options and wires those up to the
> existing flags in the PMBuilder (not the global flags). Please review!
>
> Nick
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list