This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [patch] PR ld/11817 Revert part of compressed debug patch to fix ld breakage
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Cary Coutant <ccoutant at google dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Wed, 14 Jul 2010 12:39:16 -0700
- Subject: Re: [patch] PR ld/11817 Revert part of compressed debug patch to fix ld breakage
- References: <AANLkTinkfAAIruc_Gi2GMjTpgeSAxWfG3ndVTmRa6Wtu@mail.gmail.com>
On Wed, Jul 14, 2010 at 12:02 PM, Cary Coutant <ccoutant@google.com> wrote:
> The part of my compressed debug section patch that affected
> bfd/dwarf2.c broke ld. This patch reverts that part. The original
> patch was an attempt to fix a problem where it would apply relocations
> before decompressing instead of the other way around. Since gnu ld
> doesn't yet support compressed debug sections anyway, reverting this
> patch will have no adverse effect there, but it will also leave
> addr2line unable to handle relocatable object files with compressed
> debug sections.
>
> -cary
>
> ? ? ? ?PR ld/11817
> ? ? ? ?* dwarf2.c (read_section): Revert patch for compressed debug sections.
> ? ? ? ?(find_line): Likewise.
> ? ? ? ?(read_and_uncompress_section): Remove.
>
I think it is OK to revert your own patch. As for addr2line, please open
a bug report.
BTW, I am interesting in compressed debug section support in ld.
It will be another topic.
Thanks.
--
H.J.