diff options
| author | Jason Ekstrand <[email protected]> | 2017-10-02 09:53:50 -0700 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2017-10-25 16:14:09 -0700 |
| commit | 279f8fb69cf68d05287e14f60cf67fc025643bc4 (patch) | |
| tree | aba9123b46f1a80d2ec447fac9527057b14b037a | |
| parent | afa0ddb81e693678069c630751e2091eea796937 (diff) | |
anv/pipeline: Call nir_lower_system_valaues after brw_preprocess_nir
We currently have a bug where nir_lower_system_values gets called before
nir_lower_var_copies so it will miss any system value uses which come
from a copy_var intrinsic. Moving it to after brw_preprocess_nir fixes
this problem.
Reviewed-by: Lionel Landwerlin <[email protected]>
Cc: [email protected]
| -rw-r--r-- | src/intel/vulkan/anv_pipeline.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c index 726dfcff4e4..9863ec334e3 100644 --- a/src/intel/vulkan/anv_pipeline.c +++ b/src/intel/vulkan/anv_pipeline.c @@ -188,13 +188,14 @@ anv_shader_compile_to_nir(struct anv_pipeline *pipeline, NIR_PASS_V(nir, nir_propagate_invariant); NIR_PASS_V(nir, nir_lower_io_to_temporaries, entry_point->impl, true, false); - NIR_PASS_V(nir, nir_lower_system_values); /* Vulkan uses the separate-shader linking model */ nir->info.separate_shader = true; nir = brw_preprocess_nir(compiler, nir); + NIR_PASS_V(nir, nir_lower_system_values); + if (stage == MESA_SHADER_FRAGMENT) NIR_PASS_V(nir, anv_nir_lower_input_attachments); |