v3dv/pipeline: remove pipeline->use_push_constants
authorAlejandro Piñeiro <apinheiro@igalia.com>
Fri, 26 Feb 2021 23:46:39 +0000 (00:46 +0100)
committerMarge Bot <eric+marge@anholt.net>
Mon, 22 Mar 2021 17:10:47 +0000 (17:10 +0000)
In the past we used this boolean for several things, it is really
superfluous right now.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9403>

src/broadcom/vulkan/v3dv_pipeline.c
src/broadcom/vulkan/v3dv_private.h
src/broadcom/vulkan/v3dv_uniforms.c

index 7ec4855..953066f 100644 (file)
@@ -818,7 +818,6 @@ lower_intrinsic(nir_builder *b, nir_intrinsic_instr *instr,
 
    case nir_intrinsic_load_push_constant:
       lower_load_push_constant(b, instr, pipeline);
-      pipeline->use_push_constants = true;
       return true;
 
    case nir_intrinsic_vulkan_resource_index:
index a79f7d5..78ab256 100644 (file)
@@ -1671,9 +1671,6 @@ struct v3dv_pipeline {
    /* If the pipeline should emit any of the stencil configuration packets */
    bool emit_stencil_cfg[2];
 
-   /* If the pipeline is using push constants */
-   bool use_push_constants;
-
    /* Blend state */
    struct {
       /* Per-RT bit mask with blend enables */
index c1de7ed..e0eea03 100644 (file)
@@ -343,7 +343,6 @@ v3dv_write_uniforms_wg_offsets(struct v3dv_cmd_buffer *cmd_buffer,
          break;
 
       case QUNIFORM_UNIFORM:
-         assert(pipeline->use_push_constants);
          cl_aligned_u32(&uniforms, cmd_buffer->push_constants_data[data]);
          break;