Mesa (master): r300g/compiler: Fix unsigned comparison with less than zero

Marek Olšák mareko at kemper.freedesktop.org
Sun Oct 20 23:33:07 UTC 2013


Module: Mesa
Branch: master
Commit: c948aab96ca512247124819c4a2736e7385ea95d
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c948aab96ca512247124819c4a2736e7385ea95d

Author: David Heidelberger <david.heidelberger at ixit.cz>
Date:   Mon Oct  7 18:06:11 2013 +0200

r300g/compiler: Fix unsigned comparison with less than zero

rc_find_free_temporary_list() returns signed integer
(in case of lack of free temporary registers returns -1),
so new_index in radeon_rename_regs() should be signed.

https://bugs.freedesktop.org/show_bug.cgi?id=54867

Signed-off-by: Marek Olšák <marek.olsak at amd.com>

---

 .../drivers/r300/compiler/radeon_rename_regs.c     |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/gallium/drivers/r300/compiler/radeon_rename_regs.c b/src/gallium/drivers/r300/compiler/radeon_rename_regs.c
index 7b29ece..498b88f 100644
--- a/src/gallium/drivers/r300/compiler/radeon_rename_regs.c
+++ b/src/gallium/drivers/r300/compiler/radeon_rename_regs.c
@@ -68,7 +68,7 @@ void rc_rename_regs(struct radeon_compiler *c, void *user)
 	variables = rc_get_variables(c);
 
 	for (var_ptr = variables; var_ptr; var_ptr = var_ptr->Next) {
-		unsigned new_index;
+		int new_index;
 		unsigned writemask;
 		struct rc_variable * var = var_ptr->Item;
 




More information about the mesa-commit mailing list