[Mesa-dev] [PATCH 3/3] ra: assert against unsigned underflow in q_total

Connor Abbott cwabbott0 at gmail.com
Fri Sep 5 17:59:32 PDT 2014


q_total should never go below 0 (which is why it's defined as unsigned),
and if it does, then something is seriously wrong.

Signed-off-by: Connor Abbott <cwabbott0 at gmail.com>
---
 src/mesa/program/register_allocate.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mesa/program/register_allocate.c b/src/mesa/program/register_allocate.c
index db2be5d..7faf672 100644
--- a/src/mesa/program/register_allocate.c
+++ b/src/mesa/program/register_allocate.c
@@ -434,6 +434,7 @@ decrement_q(struct ra_graph *g, unsigned int n)
       unsigned int n2_class = g->nodes[n2].class;
 
       if (n != n2 && !g->nodes[n2].in_stack) {
+         assert(g->nodes[n2].q_total >= g->regs->classes[n2_class]->q[n_class]);
 	 g->nodes[n2].q_total -= g->regs->classes[n2_class]->q[n_class];
       }
    }
-- 
1.8.5.2 (Apple Git-48)



More information about the mesa-dev mailing list