[PATCH] [DebugInfo] Add debug locations to constant SD nodes continued

Sergey Dmitrouk sdmitrouk at accesssoftek.com
Tue May 12 04:41:33 PDT 2015


In http://reviews.llvm.org/D9604#170593, @dblaikie wrote:

> main code change looks reasonable with one minor change suggested


Changed to `llvm_unreachable()` in new revision.

In http://reviews.llvm.org/D9604#170593, @dblaikie wrote:

> Do you have any stats on debug info size growth? A per-section breakdown/increase comparison of a clang bootstrap with -gmlt would be really handy.


Changes are surprisingly small.  I even wasn't sure I used correct executables during build, but it seems to be the case.

Changes of sections for `clang` binary built in debug mode in bytes (the rest of sections is unchanged):

  section        before      after       change
  .text          0x0374b513  0x0374b522  +0x000f
  .rodata        0x0095fed1  0x0095fe91  -0x0040
  .debug_info    0x1781dc27  0x1781dc41  +0x001a
  .debug_abbrev  0x001d1940  0x001d193e  -0x0002
  .debug_line    0x031745e3  0x03175800  +0x121d
  .debug_str     0x0409248e  0x040923f5  -0x0099
  .debug_ranges  0x00f5c530  0x00f5c560  +0x0030
  .debug_loc     0x00023eb8  0x00023f00  +0x0048


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9604

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list