[PATCH] D52862: DwarfDebug: Pick next location in case of missing location at block begin

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 4 11:14:03 PDT 2018


vsk added a comment.

In https://reviews.llvm.org/D52862#1255533, @MatzeB wrote:

> In https://reviews.llvm.org/D52862#1255529, @vsk wrote:
>
> > The core idea and test updates look good to me. I'm curious about the size changes: are you just measuring the size of executables, or of .o's? If it's the former, on Darwin, any size change resulting from this patch might indicate bugs.
>
>
> I measured executable sizes.


Ah, ok. Well, we know we have more work to do to eliminate codegen differences with/without debug info. We're planning on discussing this at the debug info BoF. (It's not a blocker for this patch.)

> I tried some flags of `llvm-size` or `llvm-dwarfdump` but could not figure out how to measure the size of the line tables directly...

You can look at the .dSYMs or .o's if you have those around:
$ size -m main.dSYM/Contents/Resources/DWARF/main | grep debug_line
	Section __debug_line: 60


Repository:
  rL LLVM

https://reviews.llvm.org/D52862





More information about the llvm-commits mailing list