[PATCH] D17898: Implement expansion of {s, u}{min, max} in integer legalization

[Matt Arsenault via llvm-commits]

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

LGTM


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp:1675
@@ +1674,3 @@
+    default: llvm_unreachable("invalid min/max opcode");
+    case ISD::SMAX: return std::make_pair(ISD::SETGT,  ISD::UMAX);
+    case ISD::UMAX: return std::make_pair(ISD::SETUGT, ISD::UMAX);
----------------
return on separate line


Repository:
  rL LLVM

http://reviews.llvm.org/D17898