[PATCH] D88026: [X86][SSE2] Follow up patch for D87236 to avoid the regression regarding horizontal unsigned 16 bit minimums and maximums.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 10:07:39 PDT 2020


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:39249
     SrcVT = Lo.getValueType();
-    MinPos = DAG.getNode(BinOp, DL, SrcVT, Lo, Hi);
+    Result = DAG.getNode(BinOp, DL, SrcVT, Lo, Hi);
   }
----------------
Is it worth pulling this in as well?


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:39282
+        DAG.getIntPtrConstant(0, DL));
+  }
+
----------------
clang-format this


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88026



More information about the llvm-commits mailing list