[PATCH] D28104: update TTI costs for arithmetic instructions on X86\SLM arch.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 29 03:02:49 PST 2016


RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:141
+  if (ST->isSLM()) {
+    if (ISD == ISD::MUL && LT.second == MVT::v4i32 ) {
+      if (Args.size() == 2) {
----------------
It'd be beneficial for all SSE targets to reuse the minRequiredElementSize code for better integer multiply costs - vXi64 especially but pre-SSE41 for vXi32 as well. So if you can split it from the rest of the SLM specific costs that would be useful. If not, we can try and do it in a later patch.


https://reviews.llvm.org/D28104





More information about the llvm-commits mailing list