On Monday, August 1, 2016 10:35:06 AM PDT Timothy Arceri wrote: > Cc: Kenneth Graunke <kenn...@whitecape.org> > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > index 3309f61..d1ac80a 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp > @@ -6381,7 +6381,7 @@ move_interpolation_to_top(nir_shader *nir) > instr > }; > > - for (int i = 0; i < ARRAY_SIZE(move); i++) { > + for (unsigned i = 0; i < ARRAY_SIZE(move); i++) { > if (move[i]->block != top) { > move[i]->block = top; > exec_node_remove(&move[i]->node); >
Bah, thanks. This used to be int i = 0; i < 3; i++ but I changed it to ARRAY_SIZE in response review feedback, so now it should be unsigned... Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev