[PATCH] x86: fix build when REGISTER_WARNINGS is zero
H.J. Lu
hjl.tools@gmail.com
Tue Oct 8 15:12:00 GMT 2013
On Tue, Oct 8, 2013 at 7:00 AM, Jan Beulich <JBeulich@suse.com> wrote:
> This was broken due to a misplaced "else".
>
> gas/
> 2013-10-08 Jan Beulich <jbeulich@suse.com>
>
> * tc-i386.c (check_word_reg): Remove misplaced "else".
> (check_long_reg): Restore symmetry with check_word_reg.
>
> --- 2013-10-07/gas/config/tc-i386.c
> +++ 2013-10-07/gas/config/tc-i386.c
> @@ -5434,13 +5434,10 @@ check_long_reg (void)
> return 0;
> }
> #if REGISTER_WARNINGS
> - else
> - as_warn (_("using `%s%s' instead of `%s%s' due to `%c' suffix"),
> - register_prefix,
> - (i.op[op].regs + REGNAM_EAX - REGNAM_AX)->reg_name,
> - register_prefix,
> - i.op[op].regs->reg_name,
> - i.suffix);
> + as_warn (_("using `%s%s' instead of `%s%s' due to `%c' suffix"),
> + register_prefix,
> + (i.op[op].regs + REGNAM_EAX - REGNAM_AX)->reg_name,
> + register_prefix, i.op[op].regs->reg_name, i.suffix);
> #endif
> }
> /* Warn if the r prefix on a general reg is missing. */
> @@ -5546,14 +5543,11 @@ check_word_reg (void)
> i.suffix);
> return 0;
> }
> - else
> #if REGISTER_WARNINGS
> - as_warn (_("using `%s%s' instead of `%s%s' due to `%c' suffix"),
> - register_prefix,
> - (i.op[op].regs + REGNAM_AX - REGNAM_EAX)->reg_name,
> - register_prefix,
> - i.op[op].regs->reg_name,
> - i.suffix);
> + as_warn (_("using `%s%s' instead of `%s%s' due to `%c' suffix"),
> + register_prefix,
> + (i.op[op].regs + REGNAM_AX - REGNAM_EAX)->reg_name,
> + register_prefix, i.op[op].regs->reg_name, i.suffix);
> #endif
> }
> return 1;
>
>
>
It is OK.
Thanks.
--
H.J.
More information about the Binutils
mailing list