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: Alan Modra <amodra at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Wed, 29 Jun 2016 09:32:17 -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> <CAMe9rOr237x6jQdfNJ68QM0UhgXdajB0-FL6z3=r2obYmuLgsQ at mail dot gmail dot com> <20160627133724 dot GC25465 at bubble dot grove dot modra dot org> <CAMe9rOpMs2Etq4ub18_-SxFtdNwUdwsHNg7W5VpPX2y6LK=86w at mail dot gmail dot com>
On Mon, Jun 27, 2016 at 6:47 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, Jun 27, 2016 at 6:37 AM, Alan Modra <amodra@gmail.com> wrote:
>> On Mon, Jun 27, 2016 at 05:49:00AM -0700, H.J. Lu wrote:
>>> 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.
>>> >
>>>
>>> I am going to check it in today.
>>
>> Please wait until someone OKs it. I personally don't see much point
>
> I will enable it only for Linux/x86 if I haven't heard anything by tomorrow.
>
>> in compressing debug info in relocatable object files.
>
This is what I checked in.
--
H.J.
From efd60acf2a959ff70b0190fd8b280425f12d276d Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Wed, 29 Jun 2016 09:29:39 -0700
Subject: [PATCH] Default to --enable-compressed-debug-sections=gas for
Linux/x86
--enable-compressed-debug-sections=gas added to binutils 2.26. Make it
default for Linux/x86 targets in 2.27.
* NEWS: Mention --enable-compressed-debug-sections=gas is the
default for Linux/x86 targets.
* configure.tgt (ac_default_compressed_debug_sections): Default
to yes for Linux/x86 targets.
---
gas/ChangeLog | 7 +++++++
gas/NEWS | 2 ++
gas/configure.tgt | 9 +++++++++
3 files changed, 18 insertions(+)
diff --git a/gas/ChangeLog b/gas/ChangeLog
index 4e94fd9..797e8a1 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,10 @@
+2016-06-29 H.J. Lu <hongjiu.lu@intel.com>
+
+ * NEWS: Mention --enable-compressed-debug-sections=gas is the
+ default for Linux/x86 targets.
+ * configure.tgt (ac_default_compressed_debug_sections): Default
+ to yes for Linux/x86 targets.
+
2016-06-29 Maciej W. Rozycki <macro@imgtec.com>
* write.c: Remove "libbfd.h" inclusion.
diff --git a/gas/NEWS b/gas/NEWS
index 08807f1..f5708a2 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,4 +1,6 @@
-*- text -*-
+* Default to --enable-compressed-debug-sections=gas for Linux/x86 targets.
+
* Add --no-pad-sections to stop the assembler from padding the end of output
sections up to their alignment boundary.
diff --git a/gas/configure.tgt b/gas/configure.tgt
index cda0e26..e2df659 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 ${cpu_type}-${fmt}-${os} in
+i386-elf-linux*)
+ # Default to compress DWARF debug sections for Linux/x86.
+ if test ${ac_default_compressed_debug_sections} = unset; then
+ ac_default_compressed_debug_sections=yes
+ fi
+ ;;
+esac
--
2.7.4