[PATCH] D54532: [InstructionSimplify] Add support for saturating add/sub

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 16 08:38:30 PST 2018


spatel added inline comments.


================
Comment at: test/Transforms/InstSimplify/saturating-add-sub.ll:170
+  call void @dummy_vec(<2 x i8> %y1)
+  %y2 = call <2 x i8> @llvm.usub.sat.v2i8(<2 x i8> <i8 0, i8 0>, <2 x i8> %a)
+  call void @dummy_vec(<2 x i8> %y2)
----------------
That should be "ssub" not "usub". Have a look at:
rL347060
and make sure I didn't introduce any typos while translating the tests to separate functions.


Repository:
  rL LLVM

https://reviews.llvm.org/D54532





More information about the llvm-commits mailing list