[Mesa-dev] [PATCH] Revert "radeon/llvm: enable unsafe math for graphics shaders"

Tom Stellard tom at stellard.net
Tue Feb 17 06:13:40 PST 2015


On Tue, Feb 17, 2015 at 05:15:42PM +0900, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
> 

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>

> This reverts commit 0e9cdedd2e3943bdb7f3543a3508b883b167e427.
> 
> It caused the grass to disappear in The Talos Principle.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89069
> Signed-off-by: Michel Dänzer <michel at daenzer.net>
> ---
>  src/gallium/drivers/radeon/radeon_llvm_emit.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/src/gallium/drivers/radeon/radeon_llvm_emit.c b/src/gallium/drivers/radeon/radeon_llvm_emit.c
> index 0f9dbab..624077c 100644
> --- a/src/gallium/drivers/radeon/radeon_llvm_emit.c
> +++ b/src/gallium/drivers/radeon/radeon_llvm_emit.c
> @@ -80,10 +80,6 @@ void radeon_llvm_shader_type(LLVMValueRef F, unsigned type)
>  	sprintf(Str, "%1d", llvm_type);
>  
>  	LLVMAddTargetDependentFunctionAttr(F, "ShaderType", Str);
> -
> -	if (type != TGSI_PROCESSOR_COMPUTE) {
> -		LLVMAddTargetDependentFunctionAttr(F, "unsafe-fp-math", "true");
> -	}
>  }
>  
>  static void init_r600_target()
> -- 
> 2.1.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list