[PATCH] D56869: [CodeGen][X86] Expand UADDSAT to NOT+UMIN+ADD

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with a couple of minors



================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:1927
     { ISD::USUBSAT,    MVT::v8i32,   2 }, // pmaxud + psubd
+    { ISD::UADDSAT,    MVT::v8i32,   3 }, // not + pminud + paddd
     { ISD::FSQRT,      MVT::f32,     7 }, // Haswell from http://www.agner.org/
----------------
Move this UADDSAT to be with the others above


================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:1964
     { ISD::USUBSAT,    MVT::v8i32,   6 }, // 2 x 128-bit Op + extract/insert
+    { ISD::UADDSAT,    MVT::v8i32,   8 }, // 2 x 128-bit Op + extract/insert
     { ISD::FSQRT,      MVT::f32,    14 }, // SNB from http://www.agner.org/
----------------
Move this UADDSAT to be with the others above


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

https://reviews.llvm.org/D56869