This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] add a configure option for using RELRO by default
- From: Andreas Schwab <schwab at suse dot de>
- To: Romain Geissler <romain dot geissler at amadeus dot com>
- Cc: <binutils at sourceware dot org>, <danielmicay at gmail dot com>
- Date: Tue, 29 Sep 2015 09:49:42 +0200
- Subject: Re: [PATCH] add a configure option for using RELRO by default
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot LNX dot 2 dot 10 dot 1509191709291 dot 15470 at ncegcolnx273 dot nce dot amadeus dot net>
Romain Geissler <romain.geissler@amadeus.com> writes:
> diff --git a/gold/options.h b/gold/options.h
> index 641efee..228c527 100644
> --- a/gold/options.h
> +++ b/gold/options.h
> @@ -1327,7 +1327,12 @@ class General_options
> DEFINE_bool(origin, options::DASH_Z, '\0', false,
> N_("Mark DSO to indicate that needs immediate $ORIGIN "
> "processing at runtime"), NULL);
> - DEFINE_bool(relro, options::DASH_Z, '\0', false,
> + DEFINE_bool(relro, options::DASH_Z, '\0',
> +#ifdef ENABLE_DEFAULT_RELRO
> + true,
> +#else
> + false,
> +#endif
You cannot use preprocessing directives inside macro arguments.
Andreas.
--
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."