Since I believe the index must always be >= 0, this looks correct. Reviewed-by: Ian Romanick <[email protected]>
On 10/19/2016 12:40 PM, Jordan Justen wrote: > This allows for more optimizations relating to power-of-two divisions. > > Signed-off-by: Jordan Justen <[email protected]> > --- > src/mesa/drivers/dri/i965/brw_nir_intrinsics.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_nir_intrinsics.c > b/src/mesa/drivers/dri/i965/brw_nir_intrinsics.c > index 059d14d..9ae161f 100644 > --- a/src/mesa/drivers/dri/i965/brw_nir_intrinsics.c > +++ b/src/mesa/drivers/dri/i965/brw_nir_intrinsics.c > @@ -116,7 +116,7 @@ lower_cs_intrinsics_convert_block(struct > lower_intrinsics_state *state, > uvec3.u32[2] = size[2]; > nir_ssa_def *mod_val = nir_build_imm(b, 3, 32, uvec3); > > - sysval = nir_imod(b, nir_idiv(b, local_index, div_val), mod_val); > + sysval = nir_umod(b, nir_udiv(b, local_index, div_val), mod_val); > break; > } > > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
