[PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon requested changes to this revision.
RKSimon added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:1217
+    setOperationAction(ISD::USUBSAT,   MVT::v32i8,  Legal);
+    setOperationAction(ISD::USUBSAT,   MVT::v16i16, Legal);
+
----------------
These need to use the HasInt256 ? Legal : Custom and you need to add a split256IntArith lowering for AVX1 targets (see LowerADD_SUB).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55787/new/

https://reviews.llvm.org/D55787