[PATCH] PR ld/20283: gold: Add a linker configure option --enable-relro
Mike Frysinger
vapier@gentoo.org
Wed Jun 22 10:43:00 GMT 2016
On 21 Jun 2016 19:45, H.J. Lu wrote:
> On Tue, Jun 21, 2016 at 6:06 PM, Mike Frysinger wrote:
> > On 21 Jun 2016 15:11, H.J. Lu wrote:
> >> +# Decide if -z relro should be enabled in ELF linker by default.
> >> +ac_default_ld_z_relro=unset
> >> +# Provide a configure time option to override our default.
> >> +AC_ARG_ENABLE(relro,
> >> + AS_HELP_STRING([--enable-relro],
> >> + [enable -z relro in ELF linker by default]),
> >> +[case "${enableval}" in
> >> + no) ac_default_ld_z_relro=0 ;;
> >> +esac])dnl
> >> +if test ${ac_default_ld_z_relro} = unset; then
> >> + ac_default_ld_z_relro=1
> >> +fi
> >
> > any reason to not just write it like:
> > AC_ARG_ENABLE(relro,
> > AS_HELP_STRING([--enable-relro],
> > [enable -z relro in ELF linker by default]))
> > if test "${enable_relro}" = "yes"; then
> > ac_default_ld_z_relro=1
> > else
> > ac_default_ld_z_relro=0
> > fi
> >
> > it's a bit simpler that way.
>
> I copied it from ld where ac_default_ld_z_relro is set to 1 unless
> --disable-relro is used or not a Linux target. For gold, it becomes
> unless --disable-relro is used. It is easier for me to keep both ld
> and gold similar.
except that patch isn't merged yet, so you have time to change it too
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://sourceware.org/pipermail/binutils/attachments/20160622/b7d60218/attachment.sig>
More information about the Binutils
mailing list