[PATCH] D43014: [X86][SSE] Enable SMIN/SMAX/UMIN/UMAX custom lowering for all legal types

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 10 15:17:22 PST 2018


craig.topper added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3354
 
+  // Is sign bits are zero, flip to UMIN/UMAX to SMIN/SMAX.
+  // Only do this if SMIN/SMAX is legal and UMIN/UMAX isn't.
----------------
Can we do this in reverse too? SMIN/SMAX->UMIN/UMAX?


Repository:
  rL LLVM

https://reviews.llvm.org/D43014





More information about the llvm-commits mailing list