diff options
| author | Connor Abbott <[email protected]> | 2019-11-12 15:38:46 +0100 |
|---|---|---|
| committer | Connor Abbott <[email protected]> | 2019-11-25 14:17:51 +0100 |
| commit | e7f4cadd02ca88fd3af5a396c71a33e91088228b (patch) | |
| tree | 1e8946082edd47efd58a071d12462b0fb4fae643 /src/amd/vulkan/radv_shader.c | |
| parent | 4d6676d78a4dc01a7affa5a97a24108685eae7ea (diff) | |
radv: Replace supports_spill with explict_scratch_args
The former was always true and hence dead code. We will want to
explicitly declare the ring offset register with ACO, but we also want
to declare the scratch offset too, and we can't try to disable it since
ACO also supports spilling and the determination of whether spilling has
to happen occurs well after setting up registers. So replace
supports_spill with something that will actually be used for ACO.
Reviewed-by: Samuel Pitoiset <[email protected]>
Diffstat (limited to 'src/amd/vulkan/radv_shader.c')
| -rw-r--r-- | src/amd/vulkan/radv_shader.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c index 81526c7eca0..a7253976f67 100644 --- a/src/amd/vulkan/radv_shader.c +++ b/src/amd/vulkan/radv_shader.c @@ -1118,8 +1118,7 @@ shader_variant_compile(struct radv_device *device, struct ac_llvm_compiler ac_llvm; bool thread_compiler; - if (options->supports_spill) - tm_options |= AC_TM_SUPPORTS_SPILL; + tm_options |= AC_TM_SUPPORTS_SPILL; if (device->instance->perftest_flags & RADV_PERFTEST_SISCHED) tm_options |= AC_TM_SISCHED; if (options->check_ir) @@ -1200,7 +1199,7 @@ radv_shader_variant_compile(struct radv_device *device, if (key) options.key = *key; - options.supports_spill = true; + options.explicit_scratch_args = use_aco; options.robust_buffer_access = device->robust_buffer_access; return shader_variant_compile(device, module, shaders, shader_count, shaders[shader_count - 1]->info.stage, info, |