Uros Bizjak <ubiz...@gmail.com> writes: > On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford > <rdsandif...@googlemail.com> wrote: >> gcc/ >> PR target/55204 >> * config/i386/i386.c (ix86_address_subreg_operand): Remove stack >> pointer check. >> (print_reg): Use true_regnum rather than REGNO. >> (ix86_print_operand_address): Remove SUBREG handling. > > The patch is OK for mainline and 4.7, if it passes H.J.'s tests with > -maddress-mode={short,long} on x32. > >> + unsigned int regno = true_regnum (x); > > I'd rather see the declaration at the beginning of the function.
OK, thanks, applied to both with that change. Richard