Mesa (master): nir/lower_amul: Use num_ubos/ssbos instead of recomputing it.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Aug 5 05:03:29 UTC 2020


Module: Mesa
Branch: master
Commit: 2e833b16bca66662938b139de90a1710cdc00f2b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2e833b16bca66662938b139de90a1710cdc00f2b

Author: Eric Anholt <eric at anholt.net>
Date:   Wed Jul  8 12:43:16 2020 -0700

nir/lower_amul: Use num_ubos/ssbos instead of recomputing it.

Now that num_ubos is correctly maintained, we can just trust it.  Fixes an
assertion failure in freedreno I triggered on
dEQP-GLES31.functional.ubo.random.all_per_block_buffers.1 for reasons I
don't really understand.

Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Reviewed-by: Rob Clark <robdclark at chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6179>

---

 src/compiler/nir/nir_lower_amul.c | 29 ++++++++++-------------------
 1 file changed, 10 insertions(+), 19 deletions(-)

diff --git a/src/compiler/nir/nir_lower_amul.c b/src/compiler/nir/nir_lower_amul.c
index 52a7d8a2302..7b2108cbc78 100644
--- a/src/compiler/nir/nir_lower_amul.c
+++ b/src/compiler/nir/nir_lower_amul.c
@@ -48,6 +48,8 @@
  */
 
 typedef struct {
+   nir_shader *shader;
+
    int (*type_size)(const struct glsl_type *, bool);
 
    /* Tables of UBOs and SSBOs mapping driver_location/base whether
@@ -102,7 +104,7 @@ large_ubo(lower_state *state, nir_src src)
    if (!nir_src_is_const(src))
       return state->has_large_ubo;
    unsigned idx = nir_src_as_uint(src);
-   assert(idx < state->max_slot);
+   assert(idx < state->shader->info.num_ubos);
    return state->large_ubos[idx];
 }
 
@@ -112,7 +114,7 @@ large_ssbo(lower_state *state, nir_src src)
    if (!nir_src_is_const(src))
       return state->has_large_ssbo;
    unsigned idx = nir_src_as_uint(src);
-   assert(idx < state->max_slot);
+   assert(idx < state->shader->info.num_ssbos);
    return state->large_ssbos[idx];
 }
 
@@ -232,25 +234,14 @@ nir_lower_amul(nir_shader *shader,
    assert(shader->options->has_imul24);
    assert(type_size);
 
-   /* uniforms list actually includes ubo's and ssbo's: */
-   int max_slot = 0;
-
-   nir_foreach_variable_with_modes (var, shader,
-                                    nir_var_mem_ubo | nir_var_mem_ssbo) {
-      int base = var->data.binding;
-      int size = MAX2(1, glsl_array_size(var->type));
-
-      max_slot = MAX2(max_slot, base + size);
-   }
-
-   NIR_VLA_FILL(bool, large_ubos, max_slot, 0);
-   NIR_VLA_FILL(bool, large_ssbos, max_slot, 0);
+   NIR_VLA_FILL(bool, large_ubos, shader->info.num_ubos, 0);
+   NIR_VLA_FILL(bool, large_ssbos, shader->info.num_ssbos, 0);
 
    lower_state state = {
-         .type_size = type_size,
-         .large_ubos = large_ubos,
-         .large_ssbos = large_ssbos,
-         .max_slot = max_slot,
+      .shader = shader,
+      .type_size = type_size,
+      .large_ubos = large_ubos,
+      .large_ssbos = large_ssbos,
    };
 
    /* Figure out which UBOs or SSBOs are large enough to be



More information about the mesa-commit mailing list