mismatched parentheses in reload1.c

[Paolo Bonzini]

This patch:

------------------------------------------------------------------------
r116277 | hubicka | 2006-08-21 02:00:14 +0200 (Mon, 21 Aug 2006) | 6 lines
        PR rtl-optimization/28071
        * reload1.c (reg_has_output_reload): Turn into regset.
        (reload_as_needed, forget_old_reloads_1, forget_marked_reloads,
        choose_reload_regs, emit_reload_insns): Update to new
        reg_has_output_reload.

------------------------------------------------------------------------

introduced changes in these lines of code (around line 7390 of reload1.c):

  else if (rld[r].out_reg == 0
           && rld[r].in != 0
           && ((REG_P (rld[r].in)
                && REGNO (rld[r].in) >= FIRST_PSEUDO_REGISTER
                && !REGNO_REG_SET_P (&reg_has_output_reload,
                                     REGNO (rld[r].in))
               || (REG_P (rld[r].in_reg)
                   && !REGNO_REG_SET_P (&reg_has_output_reload,
                                        REGNO (rld[r].in)))))
           && ! reg_set_p (rld[r].reg_rtx, PATTERN (insn)))

Notice how the OR is aligned with the parentheses, but in fact the 
closing parentheses on the previous line do *not* close up to the REG_P 
(only up to the REGNO_REG_SET_P).  I think something like this patch 
would be necessary:

                && !REGNO_REG_SET_P (&reg_has_output_reload,
-                                    REGNO (rld[r].in))
+                                    REGNO (rld[r].in)))
               || (REG_P (rld[r].in_reg)
                   && !REGNO_REG_SET_P (&reg_has_output_reload,
-                                       REGNO (rld[r].in)))))
+                                       REGNO (rld[r].in))))


Honza, do you agree?

Paolo