[Mesa-dev] [PATCH 4/5] radeonsi: Re-enable LLVM IR dumps

[Michel Dänzer]

On 16.01.2015 09:48, Tom Stellard wrote:
> This was inadvertently disabled by
> 761e36b4caab4e8e09a4c2b1409a825902fc7d2c.
> ---
>  src/gallium/drivers/radeon/radeon_llvm_emit.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/radeon/radeon_llvm_emit.c b/src/gallium/drivers/radeon/radeon_llvm_emit.c
> index b98afb2..0f9dbab 100644
> --- a/src/gallium/drivers/radeon/radeon_llvm_emit.c
> +++ b/src/gallium/drivers/radeon/radeon_llvm_emit.c
> @@ -162,7 +162,6 @@ unsigned radeon_llvm_compile(LLVMModuleRef M, struct radeon_shader_binary *binar
>  		strncpy(cpu, gpu_family, CPU_STRING_LEN);
>  		memset(fs, 0, sizeof(fs));
>  		if (dump) {
> -			LLVMDumpModule(M);
>  			strncpy(fs, "+DumpCode", FS_STRING_LEN);
>  		}
>  		tm = LLVMCreateTargetMachine(target, triple, cpu, fs,
> @@ -170,6 +169,9 @@ unsigned radeon_llvm_compile(LLVMModuleRef M, struct radeon_shader_binary *binar
>  						  LLVMCodeModelDefault);
>  		dispose_tm = true;
>  	}
> +	if (dump) {
> +		LLVMDumpModule(M);
> +	}
>  	/* Setup Diagnostic Handler*/
>  	llvm_ctx = LLVMGetModuleContext(M);
>  
> 

Reviewed-and-Tested-by: Michel Dänzer <michel.daenzer at amd.com>

Please push this patch now, regardless of what happens with the rest of
the series.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer