[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