[Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

[Jan Vesely]

On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> Use family, but only set xnack+ for gfx9.
> 
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
>  src/amd/common/ac_llvm_util.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/amd/common/ac_llvm_util.c b/src/amd/common/ac_llvm_util.c
> index 4f32d4b..84ed73f 100644
> --- a/src/amd/common/ac_llvm_util.c
> +++ b/src/amd/common/ac_llvm_util.c
> @@ -126,7 +126,9 @@ LLVMTargetMachineRef ac_create_target_machine(enum radeon_family family, enum ac
>  	LLVMTargetRef target = ac_get_llvm_target(triple);
>  
>  	snprintf(features, sizeof(features),
> -		 "+DumpCode,+vgpr-spilling,-fp32-denormals,-xnack");
> +		 "+DumpCode,+vgpr-spilling,-fp32-denormals%s",
> +		 family >= CHIP_VEGA10 ? ",+xnack" : ",-xnack");

isn't xnack also supported on carrizo?

Jan

> +	
>  	LLVMTargetMachineRef tm = LLVMCreateTargetMachine(
>  	                             target,
>  	                             triple,

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170706/72813f97/attachment.sig>