http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57351

--- Comment #9 from chrbr at gcc dot gnu.org ---
no, in fact, you confused me. The patch was committed correctly :-)

> On a side note, in your patch you say:
> -      t = one_reg_loc_descriptor (REGNO (XVECEXP (regs, 0, i)),
> +      reg = REGNO (XVECEXP (regs, 0, i));
> +      t = one_reg_loc_descriptor (DBX_REGISTER_NUMBER (reg),
> 
> but reg is undeclared in this scope (changing it to "unsigned reg = ..."
> fixes it)
> which suggests to me there's some other changes?
> In your original patch you also moved the #ifdef LEAF_REG_MAP and associated
> definitions inside the "if (regs == NULL_RTX)" block...

are you sure you have the good version of this file ? you should have

t = one_reg_loc_descriptor (dbx_reg_number (XVECEXP (regs, 0, i)),

so reg is not needed and doesn't need to be initialized. reg is set/used only
in the "/* Simple, contiguous registers.  */ part.

The original failing attachment now passes with the "ARM" patch. Will post
after checking with the testsuite

Regards,

Reply via email to