[PATCH] D67678: PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 23 10:28:07 PST 2020


dexonsmith added a comment.



In D67678#1836628 <https://reviews.llvm.org/D67678#1836628>, @steven_wu wrote:

> In D67678#1834957 <https://reviews.llvm.org/D67678#1834957>, @rsmith wrote:
>
> > In D67678#1834542 <https://reviews.llvm.org/D67678#1834542>, @steven_wu wrote:
> >
> > > @rsmith This also breaks macOS SDK. Can we revert this as we discuss a less aggressive option?
> >
> >
> > Do you have a timeline for how long it would take to fix the MacOS SDK? Is this something we could realistically aim to do in Clang 11 instead? I would really prefer for us to not have different defaults for Darwin versus everywhere else, so if waiting one release cycle is enough, that seems fine.
>
>
> I am also not sure if we have rules about SDK compatibility for releases and I can't say macOS SDK can definitely be fixed before clang 11 release. @dexonsmith @arphaman might have more info.


We can't comment on future releases.

That said, if this is urgent, let @arphaman know and we can try to qualify internally sometime in the next few months and we can report back what issues we find.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67678





More information about the cfe-commits mailing list