This is the mail archive of the
java@gcc.gnu.org
mailing list for the Java project.
Re: [PATCH] Set correct source location for deallocator calls
- From: Mark Wielaard <mark at klomp dot org>
- To: Dehao Chen <dehao at google dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Andrew Pinski <pinskia at gmail dot com>, Bryce McKinlay <bmckinlay at gmail dot com>, Andrew Haley <aph at redhat dot com>, Richard Henderson <rth at redhat dot com>, Jason Merrill <jason at redhat dot com>, Richard Guenther <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Li <davidxl at google dot com>, java at gcc dot gnu dot org
- Date: Sun, 16 Sep 2012 00:38:53 +0200
- Subject: Re: [PATCH] Set correct source location for deallocator calls
- References: <CALUNu-qXtr3pSTa8j60nD4Y4QK6VNAEOaPh34RRd4qt671gzkA@mail.gmail.com> <CALUNu-p2-rvGCLYn9s1gy-rGOq+cOwZCw9S1Twbjcba0dVKTfQ@mail.gmail.com> <50463661.1020303@redhat.com> <CALUNu-oP9dqG63EHyN18gYj4P3gwrrJcp4Zgrz-pABNxabmsCg@mail.gmail.com> <1346839095.9368.1.camel@springer.wildebeest.org> <CAO2gOZVhW2+VfKU8+n2qrcKr9fgM1G4bRyUTUB_0DATH9eEW6w@mail.gmail.com> <CAMe9rOqpzNa-u7b-nUfeBkpSed5Fkh16tzw4_uyptpMpKfhUfA@mail.gmail.com> <CA+=Sn1=OrqyrGCA-nY1bTpHLc68=q0QGU1t0dsZpA3meLhy0tg@mail.gmail.com> <CAMe9rOpNE0j2eGJDe9PwQg_6JevLVfKCoviVGAwUVCieUMVhFQ@mail.gmail.com> <CAO2gOZUaAjLSmOj-TfSxgVEwxgN349SgePyJaxWhnJqF7iUxuw@mail.gmail.com>
On Sun, Sep 16, 2012 at 12:09:04AM +0800, Dehao Chen wrote:
> I tried the up-to-date addr2line on any "gcc -g" generated code, it
> does not work either. This is because in the new dwarf, the
> DW_AT_high_pc now actually means the size. e.g.
>
> <1><9b>: Abbrev Number: 2 (DW_TAG_subprogram)
> <9c> DW_AT_external : 1
> <9c> DW_AT_name : bar
> <a0> DW_AT_decl_file : 1
> <a1> DW_AT_decl_line : 8
> <a2> DW_AT_linkage_name: (indirect string, offset: 0x7b): _Z3barv
> <a6> DW_AT_type : <0x8d>
> <aa> DW_AT_low_pc : 0x400583
> <b2> DW_AT_high_pc : 0x37 0x0
> <ba> DW_AT_frame_base : 1 byte block: 9c (DW_OP_call_frame_cfa)
> <bc> DW_AT_GNU_all_call_sites: 1
> <bc> DW_AT_sibling : <0xff>
>
> However, addr2line still thinks DW_AT_high_pc means "high_pc". I think
> we should wait for binutil to catch up with gcc.
I thought it had some months ago:
http://sourceware.org/ml/binutils/2012-04/msg00447.html
If that patch is present in your binutils and addr2line still doesn't
work as intented there might be a bug or some other place to update.
Thanks,
Mark