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

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp:605
   SDValue Op2 = N->getOperand(1);
-  unsigned OldBits = Op1.getValueSizeInBits();
+  unsigned OldBits = Op1.getValueType().getScalarSizeInBits();
 
----------------
Op1.getScalarValueSizeInBits();


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:26275
   case X86ISD::AVG: {
-    // Legalize types for X86ISD::AVG/ADDUS/SUBUS/VPMADDWD by widening.
+    // Legalize types for X86ISD::AVG/UADDSAT/USUBSAT/VPMADDWD by widening.
     assert(Subtarget.hasSSE2() && "Requires at least SSE2!");
----------------
X86ISD::AVG/VPMADDWD or ISD::UADDSAT/USUBSAT


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55787