[PATCH] D37534: [X86] Unsigned saturation subtraction canonicalization [the backend part]

[Yulia Koval via Phabricator via llvm-commits]

yulia_koval updated this revision to Diff 118746.
yulia_koval added a comment.

Thanks, fixed your comment in additional patch below.


https://reviews.llvm.org/D37534

Files:
  lib/Target/X86/X86ISelLowering.cpp


Index: lib/Target/X86/X86ISelLowering.cpp
===================================================================
--- lib/Target/X86/X86ISelLowering.cpp
+++ lib/Target/X86/X86ISelLowering.cpp
@@ -35922,9 +35922,9 @@
     SubusRHS = Op1;
     SDValue MaxLHS = Op0.getOperand(0);
     SDValue MaxRHS = Op0.getOperand(1);
-    if (DAG.isEqualTo(MaxLHS, Op1))
+    if (MaxLHS == Op1)
       SubusLHS = MaxRHS;
-    else if (DAG.isEqualTo(MaxRHS, Op1))
+    else if (MaxRHS == Op1)
       SubusLHS = MaxLHS;
     else
       return SDValue();
@@ -35932,9 +35932,9 @@
     SubusLHS = Op0;
     SDValue MinLHS = Op1.getOperand(0);
     SDValue MinRHS = Op1.getOperand(1);
-    if (DAG.isEqualTo(MinLHS, Op0))
+    if (MinLHS == Op0)
       SubusRHS = MinRHS;
-    else if (DAG.isEqualTo(MinRHS, Op0))
+    else if (MinRHS == Op0)
       SubusRHS = MinLHS;
     else
       return SDValue();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37534.118746.patch
Type: text/x-patch
Size: 880 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171012/0912304a/attachment.bin>