[PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 17 14:47:19 PST 2018
RKSimon added a comment.
In D55787#1333569 <https://reviews.llvm.org/D55787#1333569>, @craig.topper wrote:
> You’ll need to use an intrinsic to test the changes Simon asked for. The existing combines for X86 do their own split for AVX1.
If that is going to be tricky you can just add a Subtarget->hasInt256() limit and a TODO comment.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55787/new/
https://reviews.llvm.org/D55787
More information about the llvm-commits
mailing list