[PATCH] D55720: [Intrinsic] Signed Fixed Point Saturation Multiplication Intrinsic
Bevin Hansson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 19 08:18:41 PST 2018
ebevhan added a comment.
Nothing sticks out to me, so I think it looks good. Hard to tell if there are any sneaky edge cases in the lowering steps, though.
Maybe you should rebase this on top of the unsigned patch, since they're touching all the same places. Or are you waiting for it to land?
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55720/new/
https://reviews.llvm.org/D55720
More information about the llvm-commits
mailing list