[PATCH] D49676: [DWARF] support for .debug_addr (consumer)

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 31 14:49:54 PDT 2018


dblaikie added a comment.

In https://reviews.llvm.org/D49676#1179229, @vleschuk wrote:

> In https://reviews.llvm.org/D49676#1179057, @dblaikie wrote:
>
> > >> Also - I know for DW_AT_ranges attributes we have some advanced dumping in some situations that can print section names and the like, I think? Could we reuse that here too so the dumped addresses also have section name+offset, perhaps?
> > > 
> > > Sorry, actually I don't see this functionality for DW_AT_ranges, could you please tell me where I missed it?
> >
> > Oh, sure - it was added here: https://reviews.llvm.org/D36313
>
>
> Thank you David! If you don't mind I will implement this in a separate revision: there is already a lot on the plate for this one :-)


Yeah, better as a separate patch in any case. And there's lots of things we can improve in that regard anyway - many places addresses show up & how they could be rendered more informatively with things like this.


https://reviews.llvm.org/D49676





More information about the llvm-commits mailing list