From 7b5f2e00706899e26aaa78e595e21522ccba9c5c Mon Sep 17 00:00:00 2001 From: Dave Airlie Date: Thu, 6 Jul 2017 23:09:07 +0100 Subject: [PATCH] radv/ac: drop setting xnack Since radv uses compute rings and we can't know when we are setting up the shaders what ring they are to be used on, we should just use the default xnack setting. This may be suboptimal in some places, but if we hit a problem, we likely should try and address this between llvm and mesa. Reviewed-by: Bas Nieuwenhuizen Signed-off-by: Dave Airlie --- src/amd/common/ac_llvm_util.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/amd/common/ac_llvm_util.c b/src/amd/common/ac_llvm_util.c index d14057fa768..ea752e031ef 100644 --- a/src/amd/common/ac_llvm_util.c +++ b/src/amd/common/ac_llvm_util.c @@ -126,8 +126,7 @@ 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%s%s", - family >= CHIP_VEGA10 ? ",+xnack" : ",-xnack", + "+DumpCode,+vgpr-spilling,-fp32-denormals%s", tm_options & AC_TM_SISCHED ? ",+si-scheduler" : ""); LLVMTargetMachineRef tm = LLVMCreateTargetMachine( -- 2.30.2