From: José Fonseca <jfonseca@vmware.com>
Date: Mon, 3 Aug 2009 18:31:56 +0000 (+0100)
Subject: llvmpipe: Optimize away min/max with equal operands.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=02eaa32d83637d6f60685afe5d29aeb06345e60e;p=mesa.git

llvmpipe: Optimize away min/max with equal operands.
---

diff --git a/src/gallium/drivers/llvmpipe/lp_bld_arit.c b/src/gallium/drivers/llvmpipe/lp_bld_arit.c
index ba272df2961..db0db02c152 100644
--- a/src/gallium/drivers/llvmpipe/lp_bld_arit.c
+++ b/src/gallium/drivers/llvmpipe/lp_bld_arit.c
@@ -288,6 +288,14 @@ lp_build_intrinsic_binary(LLVMBuilderRef builder,
    }
    assert(LLVMIsDeclaration(function));
 
+#ifdef DEBUG
+   /* We shouldn't use only constants with intrinsics, as they won't be
+    * propagated by LLVM optimization passes.
+    */
+   if(LLVMIsConstant(a) && LLVMIsConstant(b))
+      debug_printf("warning: invoking intrinsic \"%s\" with constants\n");
+#endif
+
    args[0] = a;
    args[1] = b;
 
@@ -678,6 +686,9 @@ lp_build_min(struct lp_build_context *bld,
    if(a == bld->undef || b == bld->undef)
       return bld->undef;
 
+   if(a == b)
+      return a;
+
    if(bld->type.norm) {
       if(a == bld->zero || b == bld->zero)
          return bld->zero;
@@ -699,6 +710,9 @@ lp_build_max(struct lp_build_context *bld,
    if(a == bld->undef || b == bld->undef)
       return bld->undef;
 
+   if(a == b)
+      return a;
+
    if(bld->type.norm) {
       if(a == bld->one || b == bld->one)
          return bld->one;