[PATCH] add a configure option for using RELRO by default

Andreas Schwab schwab@suse.de
Tue Sep 29 07:49:00 GMT 2015


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."



More information about the Binutils mailing list