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

Thu Oct 10 01:25:24 PDT 2019

dmgreen added a subscriber: rsmith.
dmgreen accepted this revision.
dmgreen added a comment.

Yeah, OK. D67678 <https://reviews.llvm.org/D67678> is the patch that will change the default, but only to "int", not to "none" for the moment.

An earlier version of that patch had a different way of setting the default, in lib/Basic/Targets/ARM.cpp. I'm not sure that's better or worse than this, they both look fine to me.

Under that condition, LGTM!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67160