[PATCH] D68683: ARM] Fix arm_neon.h with -flax-vector-conversions=none

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

In D68683#1701769 , @rsmith wrote:

> We also have a hack in the ARM and AArch64 target info to set the default for 
> lax vector conversions back to "all" when NEON is enabled that can now be 
> removed.


Never mind, looks like I didn't get round to landing D67678 
 yet. =)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68683



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

[PATCH] D68683: ARM] Fix arm_neon.h with -flax-vector-conversions=none

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith accepted this revision.
rsmith added a comment.

Thank you!

We also have a hack in the ARM and AArch64 target info to set the default for 
lax vector conversions back to "all" when NEON is enabled that can now be 
removed.


Repository:
  rC Clang

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

https://reviews.llvm.org/D68683



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

[PATCH] D68683: ARM] Fix arm_neon.h with -flax-vector-conversions=none

[Sjoerd Meijer via Phabricator via cfe-commits]

SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Nice one, thanks for fixing! I didn't have the bandwidth to look into this.


Repository:
  rC Clang

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

https://reviews.llvm.org/D68683



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