Mesa (master): v3d: Only call nir_lower_io on shader_in/out
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon Jul 6 20:19:16 UTC 2020
Module: Mesa
Branch: master
Commit: 23b7094829d185ba2b40cdb75b730849b21e8b2b
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=23b7094829d185ba2b40cdb75b730849b21e8b2b
Author: Jason Ekstrand <jason at jlekstrand.net>
Date: Wed Jun 10 18:02:39 2020 -0500
v3d: Only call nir_lower_io on shader_in/out
Gallium drivers should never see nir_var_uniform because gallium lowers
regular uniforms to a UBO. No GL driver should ever see either
nir_var_mem_shared because that's lowered in GLSL IR.
Reviewed-by: Eric Anholt <eric at anholt.net>
Reviewed-by: Connor Abbott <cwabbott0 at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5418>
---
src/gallium/drivers/v3d/v3d_program.c | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/v3d/v3d_program.c b/src/gallium/drivers/v3d/v3d_program.c
index dbe3c7b4454..2415b279754 100644
--- a/src/gallium/drivers/v3d/v3d_program.c
+++ b/src/gallium/drivers/v3d/v3d_program.c
@@ -306,14 +306,12 @@ v3d_uncompiled_shader_create(struct pipe_context *pctx,
s = tgsi_to_nir(ir, pctx->screen, false);
}
- nir_variable_mode lower_mode = nir_var_all & ~nir_var_uniform;
- if (s->info.stage == MESA_SHADER_VERTEX ||
- s->info.stage == MESA_SHADER_GEOMETRY) {
- lower_mode &= ~(nir_var_shader_in | nir_var_shader_out);
- }
- NIR_PASS_V(s, nir_lower_io, lower_mode,
- type_size,
- (nir_lower_io_options)0);
+ if (s->info.stage != MESA_SHADER_VERTEX &&
+ s->info.stage != MESA_SHADER_GEOMETRY) {
+ NIR_PASS_V(s, nir_lower_io,
+ nir_var_shader_in | nir_var_shader_out,
+ type_size, (nir_lower_io_options)0);
+ }
NIR_PASS_V(s, nir_lower_regs_to_ssa);
NIR_PASS_V(s, nir_normalize_cubemap_coords);
More information about the mesa-commit
mailing list