This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] ld: don't use ia64 register name in expression of pr16322 test
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Jan Beulich <JBeulich at suse dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Thu, 8 Jan 2015 04:58:19 -0800
- Subject: Re: [PATCH] ld: don't use ia64 register name in expression of pr16322 test
- Authentication-results: sourceware.org; auth=none
- References: <54AE88CD02000078000529F5 at mail dot emea dot novell dot com>
On Thu, Jan 8, 2015 at 4:40 AM, Jan Beulich <JBeulich@suse.com> wrote:
> "f1" is a register name on ia64, and hence gets warned upon when used
> as expression value:
>
> .../binutils/2.25/ld/testsuite/ld-elf/pr16322.s: Assembler messages:
> .../binutils/2.25/ld/testsuite/ld-elf/pr16322.s:6: Warning: register value used as expression
>
> Change the name (and "p1" at once, which is a register name too, albeit
> not currently causing any immediate problem).
>
> ld/testsuite/
> 2015-01-08 Jan Beulich <jbeulich@suse.com>
>
> * ld-elf/pr16322.s (p1): Rename to px1.
> (f1): Rename to fx1.
>
> --- 2015-01-08/ld/testsuite/ld-elf/pr16322.s 2014-02-20 15:36:37.000000000 +0100
> +++ 2015-01-08/ld/testsuite/ld-elf/pr16322.s 2015-01-08 12:53:04.000000000 +0100
> @@ -1,6 +1,6 @@
> - .globl p1
> + .globl px1
> .section .data.rel.ro,"aw",%progbits
> .p2align 5
> - .type p1, %object
> -p1:
> - .dc.a f1
> + .type px1, %object
> +px1:
> + .dc.a fx1
>
>
>
It is OK.
Thanks.
--
H.J.