[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