i965/blorp: Rename push constants to inputs
authorTopi Pohjolainen <topi.pohjolainen@intel.com>
Tue, 17 May 2016 12:44:39 +0000 (15:44 +0300)
committerTopi Pohjolainen <topi.pohjolainen@intel.com>
Mon, 4 Jul 2016 05:37:51 +0000 (08:37 +0300)
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/mesa/drivers/dri/i965/brw_blorp.h
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
src/mesa/drivers/dri/i965/gen6_blorp.c
src/mesa/drivers/dri/i965/gen7_blorp.c

index b895e393b37400613174f0611efaa19f23e77884..a4036c1d0a9bc31be6d71e0a0c883ed5351fd69d 100644 (file)
@@ -179,7 +179,7 @@ struct brw_blorp_coord_transform
    float offset;
 };
 
-struct brw_blorp_wm_push_constants
+struct brw_blorp_wm_inputs
 {
    uint32_t dst_x0;
    uint32_t dst_x1;
@@ -201,11 +201,11 @@ struct brw_blorp_wm_push_constants
 };
 
 #define BRW_BLORP_NUM_PUSH_CONSTANT_DWORDS \
-   (sizeof(struct brw_blorp_wm_push_constants) / 4)
+   (sizeof(struct brw_blorp_wm_inputs) / 4)
 
 /* Every 32 bytes of push constant data constitutes one GEN register. */
 static const unsigned int BRW_BLORP_NUM_PUSH_CONST_REGS =
-   sizeof(struct brw_blorp_wm_push_constants) / 32;
+   sizeof(struct brw_blorp_wm_inputs) / 32;
 
 struct brw_blorp_prog_data
 {
@@ -255,7 +255,7 @@ struct brw_blorp_params
       unsigned resolve_type;
    };
    bool color_write_disable[4];
-   struct brw_blorp_wm_push_constants wm_push_consts;
+   struct brw_blorp_wm_inputs wm_inputs;
    unsigned num_draw_buffers;
    unsigned num_layers;
    uint32_t wm_prog_kernel;
index cdb6b330cdc48852bb42f8cec6e28a2488eb7a52..16d2504d1ae35d1756f28777a11bd3627d6ffb8d 100644 (file)
@@ -330,7 +330,7 @@ enum sampler_message_arg
 };
 
 struct brw_blorp_blit_vars {
-   /* Uniforms values from brw_blorp_wm_push_constants */
+   /* Input values from brw_blorp_wm_inputs */
    nir_variable *u_dst_x0;
    nir_variable *u_dst_x1;
    nir_variable *u_dst_y0;
@@ -357,7 +357,7 @@ brw_blorp_blit_vars_init(nir_builder *b, struct brw_blorp_blit_vars *v,
 #define LOAD_UNIFORM(name, type)\
    v->u_##name = nir_variable_create(b->shader, nir_var_uniform, type, #name); \
    v->u_##name->data.location = \
-      offsetof(struct brw_blorp_wm_push_constants, name);
+      offsetof(struct brw_blorp_wm_inputs, name);
 
    LOAD_UNIFORM(dst_x0, glsl_uint_type())
    LOAD_UNIFORM(dst_x1, glsl_uint_type())
@@ -1832,28 +1832,28 @@ brw_blorp_blit_miptrees(struct brw_context *brw,
    /* Round floating point values to nearest integer to avoid "off by one texel"
     * kind of errors when blitting.
     */
-   params.x0 = params.wm_push_consts.dst_x0 = roundf(dst_x0);
-   params.y0 = params.wm_push_consts.dst_y0 = roundf(dst_y0);
-   params.x1 = params.wm_push_consts.dst_x1 = roundf(dst_x1);
-   params.y1 = params.wm_push_consts.dst_y1 = roundf(dst_y1);
-   params.wm_push_consts.rect_grid_x1 =
+   params.x0 = params.wm_inputs.dst_x0 = roundf(dst_x0);
+   params.y0 = params.wm_inputs.dst_y0 = roundf(dst_y0);
+   params.x1 = params.wm_inputs.dst_x1 = roundf(dst_x1);
+   params.y1 = params.wm_inputs.dst_y1 = roundf(dst_y1);
+   params.wm_inputs.rect_grid_x1 =
       minify(src_mt->logical_width0, src_level) * wm_prog_key.x_scale - 1.0f;
-   params.wm_push_consts.rect_grid_y1 =
+   params.wm_inputs.rect_grid_y1 =
       minify(src_mt->logical_height0, src_level) * wm_prog_key.y_scale - 1.0f;
 
-   brw_blorp_setup_coord_transform(&params.wm_push_consts.x_transform,
+   brw_blorp_setup_coord_transform(&params.wm_inputs.x_transform,
                                    src_x0, src_x1, dst_x0, dst_x1, mirror_x);
-   brw_blorp_setup_coord_transform(&params.wm_push_consts.y_transform,
+   brw_blorp_setup_coord_transform(&params.wm_inputs.y_transform,
                                    src_y0, src_y1, dst_y0, dst_y1, mirror_y);
 
    if (brw->gen >= 8 && params.src.mt->target == GL_TEXTURE_3D) {
       /* On gen8+ we use actual 3-D textures so we need to pass the layer
        * through to the sampler.
        */
-      params.wm_push_consts.src_z = params.src.layer;
+      params.wm_inputs.src_z = params.src.layer;
    } else {
       /* On gen7 and earlier, we fake everything with 2-D textures */
-      params.wm_push_consts.src_z = 0;
+      params.wm_inputs.src_z = 0;
    }
 
    if (params.dst.num_samples <= 1 && dst_mt->num_samples > 1) {
index 2515a04fca1439e3a59c407dee83c290cbfbfb3c..3283a083ce6d8501e1cd9c7423521945080d1e29 100644 (file)
@@ -150,7 +150,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
       params.y1 = rb->Height - fb->_Ymin;
    }
 
-   memcpy(&params.wm_push_consts.dst_x0,
+   memcpy(&params.wm_inputs.dst_x0,
           ctx->Color.ClearColor.f, sizeof(float) * 4);
 
    bool use_simd16_replicated_data = true;
@@ -175,7 +175,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
        !partial_clear && use_simd16_replicated_data &&
        brw_is_color_fast_clear_compatible(brw, irb->mt,
                                           &ctx->Color.ClearColor)) {
-      memset(&params.wm_push_consts, 0xff, 4*sizeof(float));
+      memset(&params.wm_inputs, 0xff, 4*sizeof(float));
       params.fast_clear_op = GEN7_PS_RENDER_TARGET_FAST_CLEAR_ENABLE;
 
       brw_get_fast_clear_rect(brw, fb, irb->mt, &params.x0, &params.y0,
index eb11156dd85054b0e99e8ca93419c353999e282a..cd02b2498bb9812410af2f88d9fa85f11fcf197e 100644 (file)
@@ -279,10 +279,10 @@ gen6_blorp_emit_wm_constants(struct brw_context *brw,
    uint32_t wm_push_const_offset;
 
    uint32_t *constants = brw_state_batch(brw, AUB_TRACE_WM_CONSTANTS,
-                                         sizeof(params->wm_push_consts),
+                                         sizeof(params->wm_inputs),
                                          32, &wm_push_const_offset);
 
-   const uint32_t *push_consts = (const uint32_t *)&params->wm_push_consts;
+   const uint32_t *push_consts = (const uint32_t *)&params->wm_inputs;
    for (unsigned i = 0; i < params->wm_prog_data->nr_params; i++)
       constants[i] = push_consts[params->wm_prog_data->param[i]];
 
@@ -682,7 +682,7 @@ gen6_blorp_emit_constant_ps(struct brw_context *brw,
    /* Make sure the push constants fill an exact integer number of
     * registers.
     */
-   assert(sizeof(struct brw_blorp_wm_push_constants) % 32 == 0);
+   STATIC_ASSERT(sizeof(struct brw_blorp_wm_inputs) % 32 == 0);
 
    /* There must be at least one register worth of push constant data. */
    assert(BRW_BLORP_NUM_PUSH_CONST_REGS > 0);
index 3a8643f997239673e08c6d09ba1edd5a70350025..e4cae1f6ca10d2132cabeca403cda9a2aba5928c 100644 (file)
@@ -621,7 +621,7 @@ gen7_blorp_emit_constant_ps(struct brw_context *brw,
    /* Make sure the push constants fill an exact integer number of
     * registers.
     */
-   assert(sizeof(struct brw_blorp_wm_push_constants) % 32 == 0);
+   STATIC_ASSERT(sizeof(struct brw_blorp_wm_inputs) % 32 == 0);
 
    /* There must be at least one register worth of push constant data. */
    assert(BRW_BLORP_NUM_PUSH_CONST_REGS > 0);