yaxunl added a comment.

In D76520#1937217 <https://reviews.llvm.org/D76520#1937217>, @tra wrote:

> In D76520#1936837 <https://reviews.llvm.org/D76520#1936837>, @yaxunl wrote:
>
> > -Xarch_ works with driver options having value, e.g. 
> > `-fcf-protection=branch`. I added a test for that.
> >
> > `-mframe-pointer=none` is a cc1 option. That's why it cannot be passed by 
> > -Xarch_. If it is made a driver option it can be passed.
>
>
> In other words, if we need to pass a cc1 option to a specific 
> sub-compilation, we'll need an arch-specific `-Xclang` variant. Does 
> `-Xarch_foo -Xclang -Xarch_foo <foo-specific cc1 option>` work with this 
> patch? Considering that sub-compilations *are* cc1 invocations, passing cc1 
> options is something that would be very useful as not all cc1 options can be 
> directly controlled from the driver level, but most driver options not 
> directly related to the pipeline construction can be overridden by cc1 
> options.


-Xarch_ does not work for passing -cc1 options in the beginning. This patch 
does not change that.

This requires some further changes about how the options after -Xarch_ are 
handled. I would suggest to do that in another patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76520/new/

https://reviews.llvm.org/D76520



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to