[PATCH] D21797: [ARM] Do not test for CPUs, use SubtargetFeatures. Also remove 2 flags.

Diana Picus via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 02:19:51 PDT 2016


rovka added a comment.

In http://reviews.llvm.org/D21797#474927, @rengolin wrote:

> I personally prefer "widenVMOVS" than "dontWidenVMOVS", but otherwise, LGTM. Thanks!


Indeed, but widening seems to be the default across targets, so it would be unpleasant to add it everywhere (including for every new target). If we want to keep it on a positive tone, we could name it "KeepVMOVS" or "PreferVMOVS" (we already have a "PreferVMOVSR", I'm not sure if that makes things better or worse).


http://reviews.llvm.org/D21797





More information about the llvm-commits mailing list