[Mesa-dev] [PATCH 1/1] util: Silence signed-unsigned comparison warnings

Jan Vesely jan.vesely at rutgers.edu
Tue Dec 16 15:42:24 PST 2014


On Fri, 2014-12-12 at 17:30 +0000, Jose Fonseca wrote:
> On 11/12/14 20:05, Jan Vesely wrote:
> > Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> > ---
> >   src/util/register_allocate.c | 12 ++++++------
> >   1 file changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/src/util/register_allocate.c b/src/util/register_allocate.c
> > index 1cfd66f..af7a20c 100644
> > --- a/src/util/register_allocate.c
> > +++ b/src/util/register_allocate.c
> > @@ -79,7 +79,7 @@
> >   #include "main/bitset.h"
> >   #include "register_allocate.h"
> >
> > -#define NO_REG ~0
> > +#define NO_REG ~0U
> >
> >   struct ra_reg {
> >      BITSET_WORD *conflicts;
> > @@ -251,7 +251,7 @@ void
> >   ra_add_transitive_reg_conflict(struct ra_regs *regs,
> >   			       unsigned int base_reg, unsigned int reg)
> >   {
> > -   int i;
> > +   unsigned int i;
> >
> >      ra_add_reg_conflict(regs, reg, base_reg);
> >
> > @@ -328,7 +328,7 @@ ra_set_finalize(struct ra_regs *regs, unsigned int **q_values)
> >
> >   	 for (rc = 0; rc < regs->count; rc++) {
> >   	    int conflicts = 0;
> > -	    int i;
> > +	    unsigned int i;
> >
> >               if (!reg_belongs_to_class(rc, regs->classes[c]))
> >   	       continue;
> > @@ -481,7 +481,7 @@ ra_simplify(struct ra_graph *g)
> >   	 }
> >         }
> >
> > -      if (!progress && best_optimistic_node != ~0) {
> > +      if (!progress && best_optimistic_node != ~0U) {
> >   	 decrement_q(g, best_optimistic_node);
> >   	 g->stack[g->stack_count] = best_optimistic_node;
> >   	 g->stack_count++;
> > @@ -501,10 +501,10 @@ ra_simplify(struct ra_graph *g)
> >   static bool
> >   ra_select(struct ra_graph *g)
> >   {
> > -   int i;
> >      int start_search_reg = 0;
> >
> >      while (g->stack_count != 0) {
> > +      unsigned int i;
> >         unsigned int ri;
> >         unsigned int r = -1;
> >         int n = g->stack[g->stack_count - 1];
> > @@ -585,7 +585,7 @@ ra_set_node_reg(struct ra_graph *g, unsigned int n, unsigned int reg)
> >   static float
> >   ra_get_spill_benefit(struct ra_graph *g, unsigned int n)
> >   {
> > -   int j;
> > +   unsigned int j;
> >      float benefit = 0;
> >      int n_class = g->nodes[n].class;
> >
> >
> 
> Reviewed-by: Jose Fonseca <jfonseca at vmware.com>

thanks, can you push it as well? I don't have commit access

jan

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141216/7a15e185/attachment.sig>


More information about the mesa-dev mailing list