[PATCH] D24288: Don't reduce mul if the target doesn't support SSE2

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 7 10:17:01 PDT 2016


mkuper accepted this revision.
mkuper added a comment.
This revision is now accepted and ready to land.

LGTM, except another small change to the test.
(No need to reupload the changed version for review, feel free to commit)


================
Comment at: test/CodeGen/X86/pr30298.ll:6
@@ +5,3 @@
+
+define void @mul_2xi8(i8* nocapture readonly %a, i8* nocapture readonly %b, i64 %index) {
+; CHECK-LABEL: mul_2xi8:
----------------
Can you mark the function nounwind, so that that we don't have all the cfi clutter?


Repository:
  rL LLVM

https://reviews.llvm.org/D24288





More information about the llvm-commits mailing list