[PATCH] D67160: [clang, ARM] Default to -fno-lax-vector-conversions in ARM v8.1-M.

Simon Tatham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 9 08:44:49 PDT 2019


simon_tatham added a comment.

Yes, the commit message in rL371817 <https://reviews.llvm.org/rL371817> mentions that the long-term aim is to change the default to `none` for everyone. If that change happens before I manage to land this patch, then this patch certainly will become unnecessary.

But D68683 <https://reviews.llvm.org/D68683> doesn't make that change: it only prepares `arm_neon.h` not to fall over in future when the change does happen. And MVE needs strict vector type checking more urgently than anyone else (because for us, it's not just a nice-to-have error-checking improvement, but critical to the polymorphic ACLE intrinsics working properly in the first place).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67160





More information about the cfe-commits mailing list