[Mesa-dev] [PATCH 05/18] i965/blorp: Rename push constants to inputs
Topi Pohjolainen
topi.pohjolainen at intel.com
Thu Jun 23 19:17:01 UTC 2016
Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
---
src/mesa/drivers/dri/i965/brw_blorp.h | 8 ++++----
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 24 ++++++++++++------------
src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 4 ++--
src/mesa/drivers/dri/i965/gen6_blorp.c | 6 +++---
src/mesa/drivers/dri/i965/gen7_blorp.c | 2 +-
5 files changed, 22 insertions(+), 22 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_blorp.h b/src/mesa/drivers/dri/i965/brw_blorp.h
index b895e39..a4036c1 100644
--- a/src/mesa/drivers/dri/i965/brw_blorp.h
+++ b/src/mesa/drivers/dri/i965/brw_blorp.h
@@ -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;
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
index cdb6b33..16d2504 100644
--- a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
+++ b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
@@ -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(¶ms.wm_push_consts.x_transform,
+ brw_blorp_setup_coord_transform(¶ms.wm_inputs.x_transform,
src_x0, src_x1, dst_x0, dst_x1, mirror_x);
- brw_blorp_setup_coord_transform(¶ms.wm_push_consts.y_transform,
+ brw_blorp_setup_coord_transform(¶ms.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) {
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp b/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
index 2515a04..3283a08 100644
--- a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
+++ b/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
@@ -150,7 +150,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
params.y1 = rb->Height - fb->_Ymin;
}
- memcpy(¶ms.wm_push_consts.dst_x0,
+ memcpy(¶ms.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(¶ms.wm_push_consts, 0xff, 4*sizeof(float));
+ memset(¶ms.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, ¶ms.x0, ¶ms.y0,
diff --git a/src/mesa/drivers/dri/i965/gen6_blorp.c b/src/mesa/drivers/dri/i965/gen6_blorp.c
index eb11156..cd02b24 100644
--- a/src/mesa/drivers/dri/i965/gen6_blorp.c
+++ b/src/mesa/drivers/dri/i965/gen6_blorp.c
@@ -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 *)¶ms->wm_push_consts;
+ const uint32_t *push_consts = (const uint32_t *)¶ms->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);
diff --git a/src/mesa/drivers/dri/i965/gen7_blorp.c b/src/mesa/drivers/dri/i965/gen7_blorp.c
index 3a8643f..e4cae1f 100644
--- a/src/mesa/drivers/dri/i965/gen7_blorp.c
+++ b/src/mesa/drivers/dri/i965/gen7_blorp.c
@@ -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);
--
2.5.5
More information about the mesa-dev
mailing list