[PATCH] Add intrinsics and SDNodes for umin, umax, smin and smax.

Gerolf Hoflehner ghoflehner at apple.com
Mon Apr 27 11:55:20 PDT 2015


REPOSITORY
  rL LLVM

================
Comment at: docs/LangRef.rst:9569
@@ +9568,3 @@
+    folding will not do as good a job with these intrinsics as with
+    the canonical, and recommended, ``icmp; select`` sequence given below.
+
----------------
Is there are way to distinguish between user and compiler generated intrinsic? And assert on user?

================
Comment at: lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp:77
@@ -74,1 +76,3 @@
+  case ISD::UMIN:
+  case ISD::UMAX:        Res = PromoteIntRes_MINMAX(N, N->getOpcode()); break;
   case ISD::SHL:         Res = PromoteIntRes_SHL(N); break;
----------------
just MINMAX(N) would be consistent with (many :-) other  Promote... interfaces.

http://reviews.llvm.org/D9293

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list