st/radeonsi: enable uniform packing in NIR backend
authorTimothy Arceri <tarceri@itsqueeze.com>
Tue, 13 Mar 2018 22:51:23 +0000 (09:51 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Tue, 20 Mar 2018 03:19:35 +0000 (14:19 +1100)
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/gallium/drivers/radeonsi/si_get.c
src/mesa/state_tracker/st_glsl_to_nir.cpp

index 323700d425076de065180b1d39235877b4910f1f..b4ca5bea943d92e5812f20172a0cc71f3d36fff4 100644 (file)
@@ -251,6 +251,11 @@ static int si_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
                        return RADEON_SPARSE_PAGE_SIZE;
                return 0;
 
+       case PIPE_CAP_PACKED_UNIFORMS:
+               if (sscreen->debug_flags & DBG(NIR))
+                       return 1;
+               return 0;
+
        /* Unsupported features. */
        case PIPE_CAP_BUFFER_SAMPLER_VIEW_RGBA_ONLY:
        case PIPE_CAP_TGSI_FS_COORD_ORIGIN_LOWER_LEFT:
@@ -269,7 +274,6 @@ static int si_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
        case PIPE_CAP_TILE_RASTER_ORDER:
        case PIPE_CAP_MAX_COMBINED_SHADER_OUTPUT_RESOURCES:
        case PIPE_CAP_CONTEXT_PRIORITY_MASK:
-       case PIPE_CAP_PACKED_UNIFORMS:
                return 0;
 
        case PIPE_CAP_FENCE_SIGNAL:
index 9006650517b46209914d9e04c121a59cd246c51d..2eb2ece5b164be9b6a6bb455a0f45ad4a5f61d96 100644 (file)
@@ -753,14 +753,8 @@ st_finalize_nir(struct st_context *st, struct gl_program *prog,
    st_nir_assign_uniform_locations(st->ctx, prog, shader_program,
                                    &nir->uniforms, &nir->num_uniforms);
 
-   /* Below is a quick hack so that uniform lowering only runs on radeonsi
-    * (the only NIR backend that currently supports tess) once we enable
-    * uniform packing support we will just use
-    * ctx->Const.PackedDriverUniformStorage for this check.
-    */
-   if (screen->get_shader_param(screen, PIPE_SHADER_TESS_CTRL,
-                                PIPE_SHADER_CAP_MAX_INSTRUCTIONS) > 0) {
-      NIR_PASS_V(nir, nir_lower_io, nir_var_uniform, type_size,
+   if (st->ctx->Const.PackedDriverUniformStorage) {
+      NIR_PASS_V(nir, nir_lower_io, nir_var_uniform, st_glsl_type_dword_size,
                  (nir_lower_io_options)0);
       NIR_PASS_V(nir, st_nir_lower_uniforms_to_ubo, prog->Parameters);
    }