On Mon, Nov 5, 2012 at 10:56 AM, Richard Sandiford <[email protected]> wrote: > Uros Bizjak <[email protected]> writes: >> On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford >> <[email protected]> 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
On 4.7 branch, this failed to build x32 run-time library: /tmp/ccID3vr6.s: Assembler messages: /tmp/ccID3vr6.s:2788: Error: bad register name `%rr15' There is an extra 'r' in register name. -- H.J.
