[Mesa-dev] [PATCH] i965: Use GEN_GEN and GEN_IS_HASWELL in genX_state_upload.c code.
Eduardo Lima Mitev
elima at igalia.com
Fri Aug 25 07:13:00 UTC 2017
Reviewed-by: Eduardo Lima Mitev <elima at igalia.com>
Thanks, Ken
Eduardo
On 08/25/2017 08:40 AM, Kenneth Graunke wrote:
> We were using brw->gen, brw->is_haswell, and devinfo->gen in a few
> places, when we could just use GEN_GEN and GEN_IS_HASWELL, which are
> evaluated at compile time.
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index f1e9fa38ffc..75936632605 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -563,7 +563,7 @@ genX(emit_vertices)(struct brw_context *brw)
> * vertex element may poke over the end of the buffer by 2 bytes.
> */
> const unsigned padding =
> - (GEN_GEN <= 7 && !brw->is_baytrail && !brw->is_haswell) * 2;
> + (GEN_GEN <= 7 && !GEN_IS_HASWELL && !brw->is_baytrail) * 2;
> const unsigned end = buffer->offset + buffer->size + padding;
> dw = genX(emit_vertex_buffer_state)(brw, dw, i, buffer->bo,
> buffer->offset,
> @@ -4191,7 +4191,7 @@ genX(upload_cs_state)(struct brw_context *brw)
> .BindingTablePointer = stage_state->bind_bo_offset,
> .ConstantURBEntryReadLength = cs_prog_data->push.per_thread.regs,
> .NumberofThreadsinGPGPUThreadGroup = cs_prog_data->threads,
> - .SharedLocalMemorySize = encode_slm_size(devinfo->gen,
> + .SharedLocalMemorySize = encode_slm_size(GEN_GEN,
> prog_data->total_shared),
> .BarrierEnable = cs_prog_data->uses_barrier,
> #if GEN_GEN >= 8 || GEN_IS_HASWELL
> @@ -4682,9 +4682,9 @@ genX(upload_default_color)(struct brw_context *brw,
> color.ui[3] = float_as_int(1.0);
>
> int alignment = 32;
> - if (brw->gen >= 8) {
> + if (GEN_GEN >= 8) {
> alignment = 64;
> - } else if (brw->is_haswell && (is_integer_format || is_stencil_sampling)) {
> + } else if (GEN_IS_HASWELL && (is_integer_format || is_stencil_sampling)) {
> alignment = 512;
> }
>
>
More information about the mesa-dev
mailing list