r300/compiler: Don't use rc_error() unless the error is unrecoverable
authorTom Stellard <tstellar@gmail.com>
Tue, 28 Sep 2010 16:57:18 +0000 (09:57 -0700)
committerTom Stellard <tstellar@gmail.com>
Wed, 29 Sep 2010 06:52:41 +0000 (23:52 -0700)
https://bugs.freedesktop.org/show_bug.cgi?id=30416

NOTE: This is a candidate for the 7.9 branch.

src/mesa/drivers/dri/r300/compiler/radeon_emulate_loops.c

index cd4fcbabb938d1f22677cf2b736150bb0292db81..02a52f92fc321b9736731f3a937709d071dd712c 100644 (file)
@@ -386,8 +386,6 @@ static int build_loop_info(struct radeon_compiler * c, struct loop_info * loop,
                        case RC_OPCODE_SNE:
                                break;
                        default:
-                               rc_error(c, "%s: expected conditional",
-                                                               __FUNCTION__);
                                return 0;
                        }
                        loop->Cond = loop->If->Prev;
@@ -431,8 +429,10 @@ static int transform_loop(struct emulate_loop_state * s,
 
        loop = &s->Loops[s->LoopCount++];
 
-       if (!build_loop_info(s->C, loop, inst))
+       if (!build_loop_info(s->C, loop, inst)) {
+               rc_error(s->C, "Failed to build loop info\n");
                return 0;
+       }
 
        if(try_unroll_loop(s->C, loop)){
                return 1;