r300/compiler: Clean up rc_pair_alloc_source()

diff --git a/src/mesa/drivers/dri/r300/compiler/radeon_program_pair.c b/src/mesa/drivers/dri/r300/compiler/radeon_program_pair.c
index 5a50584b7250a37dcc0f0c2081c18a44e586f43e..c31efdb059058f863cbed993dd995cd0a1f3f738 100644 (file)
--- a/src/mesa/drivers/dri/r300/compiler/radeon_program_pair.c
+++ b/src/mesa/drivers/dri/r300/compiler/radeon_program_pair.c
@@ -45,23 +45,16 @@ int rc_pair_alloc_source(struct rc_pair_instruction *pair,
        if ((!rgb && !alpha) || file == RC_FILE_NONE)
                return 0;
 
-       if (rgb && pair->RGB.Src[RC_PAIR_PRESUB_SRC].Used) {
-               if (file == RC_FILE_PRESUB) {
-                       if (index != pair->RGB.Src[RC_PAIR_PRESUB_SRC].Index) {
+       /* Make sure only one presubtract operation is used per instruction. */
+       if (file == RC_FILE_PRESUB) {
+               if (rgb && pair->RGB.Src[RC_PAIR_PRESUB_SRC].Used
+                       && index != pair->RGB.Src[RC_PAIR_PRESUB_SRC].Index) {
                                return -1;
-                       }
-               } else {
-                       rgb_used++;
                }
-       }
 
-       if (alpha && pair->Alpha.Src[RC_PAIR_PRESUB_SRC].Used) {
-               if (file == RC_FILE_PRESUB) {
-                       if (index != pair->Alpha.Src[RC_PAIR_PRESUB_SRC].Index) {
+               if (alpha && pair->Alpha.Src[RC_PAIR_PRESUB_SRC].Used
+                       && index != pair->Alpha.Src[RC_PAIR_PRESUB_SRC].Index) {
                                return -1;
-                       }
-               } else {
-                       alpha_used++;
                }
        }
 
@@ -92,17 +85,16 @@ int rc_pair_alloc_source(struct rc_pair_instruction *pair,
                        candidate = i;
                }
        }
-       if (candidate < 0 || (rgb && rgb_used > 2) || (alpha && alpha_used > 2))
+
+       if (file == RC_FILE_PRESUB) {
+               candidate = RC_PAIR_PRESUB_SRC;
+       } else if (candidate < 0 || (rgb && rgb_used > 2)
+                       || (alpha && alpha_used > 2)) {
                return -1;
+       }
 
        /* candidate >= 0 */
 
-       /* Even if we have a presub src, the above loop needs to run,
-        * because we still need to make sure there is a free source.
-        */
-       if (file == RC_FILE_PRESUB)
-               candidate = RC_PAIR_PRESUB_SRC;
-
        if (rgb) {
                pair->RGB.Src[candidate].Used = 1;
                pair->RGB.Src[candidate].File = file;