[PATCH] D56869: [CodeGen][X86] Expand UADDSAT to NOT+UMIN+ADD
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 28 10:36:11 PST 2019
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
More information about the llvm-commits
mailing list