[PATCH] D59006: [x86] improve the default expansion of uaddsat/usubsat

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 21 08:47:42 PDT 2019


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

LGTM - thanks!



================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:23891
+  SDValue X = Op.getOperand(0);
+  SDValue Y = Op.getOperand(1);
   if (VT.getScalarType() == MVT::i1) {
----------------
Move these down inside the "if (VT.is128BitVector())" loop ?


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

https://reviews.llvm.org/D59006





More information about the llvm-commits mailing list