This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: ld -z options
On Thu, Jul 14, 2011 at 04:06:03AM -0400, Hans-Peter Nilsson wrote:
> Talking to myself...
>
> On Thu, 14 Jul 2011, Hans-Peter Nilsson wrote:
> > This caused several test-cases to go from PASS to UNRESOLVED for
> > cris-axis-linux-gnu
>
> It seems this would happen for e.g. ia64-*-linux-gnu too.
Yes.
> > All preceded by a linker warning ".../ld-new: warning: -z relro
> > ignored." in ld.log. The UNRESOLVED status then cause the
> > check-ld to return error. According to the dejagnu docs, the
> > UNRESOLVED is an indication of an error in the testsuite (likely
> > the unexpected warning).
>
> That's a bug in ld-elf/binutils.exp or the new warning message;
> binutils.exp tests [string match "*not supported*" $link_output]
> || [string match "*unrecognized option*" $link_output]
> which doesn't match "warning: -z relro ignored.".
>
> Which one should be adjusted?
Probably the new one. Patch preapproved if you'd like to do it.
> > Hm, at a second glance many other ports set
> > COMMONPAGESIZE="CONSTANT (COMMONPAGESIZE)" which for some reason
> > triggers the recognition of e.g. "-z relro" but why would I have
> > to set that when there's just one page-size?
>
> This seems the wrong default, and emulparams/elf64_ia64.sh
> should set something to disable -z relro instead of everyone
> else setting some vaguely related variable. Or is that's too
> linux-centric? Maybe create a ${srcdir}/emulparams/linux-common.sh
> for emulparams/*.sh to include?
I had the same idea, but see my other mail about bfd backend and ld
needing to agree.
--
Alan Modra
Australia Development Lab, IBM