This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] Default to --enable-compressed-debug-sections=gas for Linux
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Binutils <binutils at sourceware dot org>
- Date: Mon, 27 Jun 2016 05:49:00 -0700
- Subject: Re: [PATCH] Default to --enable-compressed-debug-sections=gas for Linux
- Authentication-results: sourceware.org; auth=none
- References: <20160624143408 dot GA17513 at intel dot com>
On Fri, Jun 24, 2016 at 7:34 AM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> --enable-compressed-debug-sections=gas added to binutils 2.26. Make it
> default for Linux in 2.27.
>
> Any comments?
>
> H.J.
> ---
> * NEWS: Mention --enable-compressed-debug-sections=gas default.
> * configure.tgt (ac_default_compressed_debug_sections): Default
> to yes for all Linux targets.
> ---
> gas/NEWS | 3 +++
> gas/configure.tgt | 9 +++++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/gas/NEWS b/gas/NEWS
> index a099060..1239cbb 100644
> --- a/gas/NEWS
> +++ b/gas/NEWS
> @@ -1,4 +1,7 @@
> -*- text -*-
> +* Default to --enable-compressed-debug-sections=gas for all Linux
> + targets.
> +
> * Support for the ARMv8-M architecture has been added to the ARM port. Support
> for the ARMv8-M Security and DSP Extensions has also been added to the ARM
> port.
> diff --git a/gas/configure.tgt b/gas/configure.tgt
> index cda0e26..d77abc4 100644
> --- a/gas/configure.tgt
> +++ b/gas/configure.tgt
> @@ -502,3 +502,12 @@ if test $bfd_gas != yes; then
> echo This target is no longer supported in gas
> exit 1
> fi
> +
> +case ${generic_target} in
> +*-*-linux*)
> + # Default to compress DWARF debug sections for Linux.
> + if test ${ac_default_compressed_debug_sections} = unset; then
> + ac_default_compressed_debug_sections=yes
> + fi
> + ;;
> +esac
> --
> 2.5.5
>
I am going to check it in today.
--
H.J.