[PATCH] D66005: [ARM] Add MVE beats vector cost model

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 9 05:34:55 PDT 2019


samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp:396
+    // We say MVE moves costs at least the MVEVectorCostFactor, even though
+    // they are scalar instructions. This helps prevent mixing scalar and
+    // vector.
----------------
What part of the architecture suggests that we should avoid mixing scalar and vector?


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

https://reviews.llvm.org/D66005





More information about the llvm-commits mailing list