Re: PATCH: Compress debug sections for Linux/x86 by default

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Sun, Dec 14, 2014 at 6:56 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> I checked in this patch to compress debug sections for Linux/x86 by
>> default.
>
> Has this been discussed before, or is this something obviously
> part of a general direction that was agreed on?

Not on binutils mailing list.  I have been using it for several years
and mentioned it in

https://sourceware.org/ml/libc-alpha/2011-06/msg00041.html

---
Starting from the 2.21.51.0.2 release, binutils fully supports compressed
debug sections.  However, compressed debug section isn't turned on by
default in assembler. I am planning to turn it on for x86 assembler in
the future release, which may lead to the Linux kernel bug messages like

WARNING: lib/ts_kmp.o (.zdebug_aranges): unexpected non-allocatable section.

But the resulting kernel works fine.
----

> Are things still going to work if GDB is configured without zlib,
> or instance?

No.  Zlib must be enabled.  Since I only enabled it in Linux/x86
assembler, I think it is acceptable.

> This is also the type of change that I think is worthy of a NEWS
> entry, IMO.
>

Good idea.  Done.

Thanks.


-- 
H.J.
---
diff --git a/gas/ChangeLog b/gas/ChangeLog
index 27da2cc..b53fcff 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -2,6 +2,7 @@

  * config/tc-i386.c (flag_compress_debug): Default to compress
  debug sections for Linux.
+ * NEWS: Mention it.

 2014-12-12  Alan Modra  <amodra@gmail.com>

diff --git a/gas/NEWS b/gas/NEWS
index 8a757c5..16035c6 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,5 +1,9 @@
 -*- text -*-

+Changes in 2.26:
+
+* --compress-debug-sections is turned on for Linux/x86 by default.
+
 Changes in 2.25:

 * Add support for the AVR Tiny microcontrollers.