[PATCH] D27756: [X86][SSE] Improve lowering of vXi64 multiplies

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 09:30:55 PST 2016


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

LGTM with one comment.



================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:556
     // Because we believe v4i64 to be a legal type, we must also include the
     // split factor of two in the cost table. Therefore, the cost here is 18
     // instead of 9.
----------------
This comment still says the cost is 18.


Repository:
  rL LLVM

https://reviews.llvm.org/D27756





More information about the llvm-commits mailing list