On Thu, Oct 22, 2015 at 11:36 AM, Kristian Høgsberg Kristensen <k...@bitplanet.net> wrote: > The emit_untyped_read and emit_untyped_write helpers already uniformize > the surface index argument. No need to do it before calling them. > > Signed-off-by: Kristian Høgsberg Kristensen <k...@bitplanet.net> > --- > src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > index 5dc63c9..00f200a 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > @@ -1511,7 +1511,6 @@ fs_visitor::nir_emit_intrinsic(const fs_builder &bld, > nir_intrinsic_instr *instr > surf_index = vgrf(glsl_type::uint_type); > bld.ADD(surf_index, get_nir_src(instr->src[0]), > fs_reg(stage_prog_data->binding_table.ssbo_start)); > - surf_index = bld.emit_uniformize(surf_index);
Looks correct -- I see that emit_send() does call uniformize. But when this was added in commit b234537, it apparently fixed piglit tests. Presumably we gained an additional uniformize in the transition to the builder infrastructure, making this one no longer necessary? If that's correct, Reviewed-by: Matt Turner <matts...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
