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

[Alex Deucher]

On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely <jan.vesely at rutgers.edu> wrote:
> 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?

Only on memory mapped via ATC which we only use with ROCm at the moment.

Alex

>
> Jan
>
>> +
>>       LLVMTargetMachineRef tm = LLVMCreateTargetMachine(
>>                                    target,
>>                                    triple,
>
> --
> Jan Vesely <jan.vesely at rutgers.edu>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>