From 02eaa32d83637d6f60685afe5d29aeb06345e60e Mon Sep 17 00:00:00 2001 From: José Fonseca Date: Mon, 3 Aug 2009 19:31:56 +0100 Subject: llvmpipe: Optimize away min/max with equal operands. --- src/gallium/drivers/llvmpipe/lp_bld_arit.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'src') diff --git a/src/gallium/drivers/llvmpipe/lp_bld_arit.c b/src/gallium/drivers/llvmpipe/lp_bld_arit.c index ba272df296..db0db02c15 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; -- cgit v1.2.3