On Thu, 2015-10-29 at 00:50 -0700, Jordan Justen wrote:
> Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com>
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 6 ++++--
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 6 ++++--
> 2 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> index 4950ba4..6d69e96 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> @@ -1438,8 +1438,10 @@ fs_visitor::nir_emit_intrinsic(const fs_builder &bld,
> nir_intrinsic_instr *instr
> fs_reg surf_index;
>
> if (const_index) {
> - surf_index = fs_reg(stage_prog_data->binding_table.ubo_start +
> - const_index->u[0]);
> + unsigned index = stage_prog_data->binding_table.ubo_start +
> + const_index->u[0];
> + surf_index = fs_reg(index);
> + brw_mark_surface_used(prog_data, index);
> } else {
> /* The block index is not a constant. Evaluate the index expression
> * per-channel and add the base UBO index; we have to select a value
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> index 0f04f65..efbdaa9 100644
> --- a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> @@ -749,8 +749,10 @@ vec4_visitor::nir_emit_intrinsic(nir_intrinsic_instr
> *instr)
> /* The block index is a constant, so just emit the binding table
> entry
> * as an immediate.
> */
> - surf_index = src_reg(prog_data->base.binding_table.ubo_start +
> - const_block_index->u[0]);
> + unsigned index = prog_data->base.binding_table.ubo_start +
> + const_block_index->u[0];
> + surf_index = src_reg(index);
> + brw_mark_surface_used(&prog_data->base, index);
> } else {
> /* The block index is not a constant. Evaluate the index expression
> * per-channel and add the base UBO index; we have to select a value
Is this necessary? As far as I can see, the generator opcodes will mark
surfaces for which we have a constant index as used, so this should not
be necessary.
Iago
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
