[PATCH] D12030: [CostModel][ARM] Increase cost of insert/extract operations

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 14 07:45:38 PDT 2015


rengolin accepted this revision.
rengolin added a reviewer: rengolin.
rengolin added a comment.
This revision is now accepted and ready to land.

LGTM. Thanks!


================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:266
@@ +265,3 @@
+    // Even if it's not a cross class copy, this likely leads to mixing
+    // of NEON and VFP code and should be therefore penalized.
+    if (ValTy->isVectorTy() &&
----------------
Excellent! I meant to look at that 2.5 years ago! :)


http://reviews.llvm.org/D12030





More information about the llvm-commits mailing list