[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