wolfgangp (Wolfgang Pieb)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 17 2014, 5:52 PM (203 w, 6 d)

Recent Activity

Tue, Jul 31

wolfgangp committed rL338437: [DWARF] Do not create a .debug_ranges section when no ranges are needed..
[DWARF] Do not create a .debug_ranges section when no ranges are needed.
Tue, Jul 31, 1:57 PM
wolfgangp closed D50089: [DWARF v4] Suppressing the __debug_ranges section when there are no ranges.
Tue, Jul 31, 1:57 PM
wolfgangp committed rL338424: [DWARF] Change a test to ensure the creation of a __debug_ranges section..
[DWARF] Change a test to ensure the creation of a __debug_ranges section.
Tue, Jul 31, 12:38 PM
wolfgangp created D50089: [DWARF v4] Suppressing the __debug_ranges section when there are no ranges.
Tue, Jul 31, 11:09 AM

Mon, Jul 30

wolfgangp added inline comments to D49964: [DWARF] Refactor DWARF classes to use unified error reporting. NFC..
Mon, Jul 30, 9:20 AM · debug-info

Thu, Jul 26

wolfgangp committed rL338081: [DWARF v5] Reposting r337981, which was reverted in r337997 due to a test….
[DWARF v5] Reposting r337981, which was reverted in r337997 due to a test…
Thu, Jul 26, 3:49 PM

Wed, Jul 25

wolfgangp committed rL337993: [DWARF v5] Don't report an error when the .debug_rnglists section is empty or….
[DWARF v5] Don't report an error when the .debug_rnglists section is empty or…
Wed, Jul 25, 6:12 PM
wolfgangp closed D49815: [DWARF v5] Don't report an error when the .debug_rnglists section is empty or non-existent (PR 38297).
Wed, Jul 25, 6:12 PM
wolfgangp committed rL337981: [DWARF v5] Don't emit multiple DW_AT_rnglists_base attributes. Some refactoring….
[DWARF v5] Don't emit multiple DW_AT_rnglists_base attributes. Some refactoring…
Wed, Jul 25, 4:03 PM
wolfgangp closed D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Wed, Jul 25, 4:03 PM
wolfgangp created D49815: [DWARF v5] Don't report an error when the .debug_rnglists section is empty or non-existent (PR 38297).
Wed, Jul 25, 11:56 AM

Tue, Jul 24

wolfgangp updated the diff for D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
  • Elaborated more on the rationale of the test case rnglists_base_attr.ll and used a simpler test case suggested by Dave.
  • adjusted the spelling of some dump strings in a test.
Tue, Jul 24, 3:39 PM
wolfgangp added a comment to D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.

Is this still missing test coverage for the case where the CU has low/high_pc, but also needs a ranges_base because a function has ranges for a local scope? Looks like the bug got fixed, but test coverage is still missing?

Tue, Jul 24, 3:34 PM

Mon, Jul 23

wolfgangp committed rL337770: Embed a template specialization in a namespace to work around a gcc bug..
Embed a template specialization in a namespace to work around a gcc bug.
Mon, Jul 23, 4:16 PM
wolfgangp added a comment to D49500: [DWARF v5] Refactor range lists dumping (NFC except for spelling changes in dump output)..

Thanks for the review, Jonas. I included the change for the DumpOpts parameter.

Mon, Jul 23, 3:38 PM
wolfgangp committed rL337763: [DWARF v5] Refactor range lists dumping by using a more generic way of handling….
[DWARF v5] Refactor range lists dumping by using a more generic way of handling…
Mon, Jul 23, 3:37 PM
wolfgangp closed D49500: [DWARF v5] Refactor range lists dumping (NFC except for spelling changes in dump output)..
Mon, Jul 23, 3:37 PM

Fri, Jul 20

wolfgangp updated the diff for D49500: [DWARF v5] Refactor range lists dumping (NFC except for spelling changes in dump output)..
  • Used StringRef instead of const char * for some strings displayed during dumping.
  • Formatting
Fri, Jul 20, 11:53 AM

Thu, Jul 19

wolfgangp added inline comments to D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Thu, Jul 19, 7:03 PM
wolfgangp updated the diff for D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.

Addressed review comments wrt correctly adding DW_AT_rnglists_base and associated test cases. More details inline.

Thu, Jul 19, 7:03 PM
wolfgangp added inline comments to D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Thu, Jul 19, 11:35 AM
wolfgangp updated the diff for D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.

Review comments: Have emitRnglistsTableHeader() return the end symbol instead of setting it as a reference parameter.

Thu, Jul 19, 11:35 AM

Wed, Jul 18

wolfgangp created D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Wed, Jul 18, 4:44 PM
wolfgangp created D49500: [DWARF v5] Refactor range lists dumping (NFC except for spelling changes in dump output)..
Wed, Jul 18, 11:22 AM

Mon, Jul 16

wolfgangp added inline comments to D49214: [DWARF v5] emit DWARF v5 range lists (no support for fission yet).
Mon, Jul 16, 12:00 PM

Jul 12 2018

wolfgangp committed rL336927: [DWARF v5] Generate range list tables into the .debug_rnglists section. No….
[DWARF v5] Generate range list tables into the .debug_rnglists section. No…
Jul 12 2018, 11:23 AM
wolfgangp closed D49214: [DWARF v5] emit DWARF v5 range lists (no support for fission yet).
Jul 12 2018, 11:23 AM
wolfgangp added a comment to D49214: [DWARF v5] emit DWARF v5 range lists (no support for fission yet).

Thanks for the quick review.

Jul 12 2018, 10:41 AM

Jul 11 2018

wolfgangp created D49214: [DWARF v5] emit DWARF v5 range lists (no support for fission yet).
Jul 11 2018, 6:38 PM

Jul 9 2018

wolfgangp committed rL336621: [DWARF][NFC] Refactor range list emission to use a static helper.
[DWARF][NFC] Refactor range list emission to use a static helper
Jul 9 2018, 5:15 PM
wolfgangp closed D49098: [DWARF][NFC] Factoring out emission of a single rangelist into a static helper.
Jul 9 2018, 5:15 PM
wolfgangp created D49098: [DWARF][NFC] Factoring out emission of a single rangelist into a static helper.
Jul 9 2018, 1:02 PM

Jun 20 2018

wolfgangp committed rL335188: [DWARF] Improved error reporting for range lists. .
[DWARF] Improved error reporting for range lists.
Jun 20 2018, 4:01 PM
wolfgangp closed D48344: [DWARF] Improved error reporting for range lists.
Jun 20 2018, 4:01 PM
wolfgangp added inline comments to D48344: [DWARF] Improved error reporting for range lists.
Jun 20 2018, 12:44 PM

Jun 19 2018

wolfgangp created D48344: [DWARF] Improved error reporting for range lists.
Jun 19 2018, 5:33 PM

May 18 2018

wolfgangp committed rL332772: Fixing buildbot error introduced with r332759..
Fixing buildbot error introduced with r332759.
May 18 2018, 2:51 PM
wolfgangp committed rL332766: Addressing a couple of compiler warnings introduced with r332759..
Addressing a couple of compiler warnings introduced with r332759.
May 18 2018, 1:55 PM
wolfgangp committed rL332762: Fixing build error introduced with r332759..
Fixing build error introduced with r332759.
May 18 2018, 1:39 PM
wolfgangp committed rL332759: [DWARF v5] Improved support for .debug_rnglists (consumer). Enables any….
[DWARF v5] Improved support for .debug_rnglists (consumer). Enables any…
May 18 2018, 1:17 PM
wolfgangp closed D45549: [DWARF v5] improved support for .debug_rnglists/consumer.
May 18 2018, 1:16 PM
wolfgangp added inline comments to D45549: [DWARF v5] improved support for .debug_rnglists/consumer.
May 18 2018, 9:47 AM

May 16 2018

wolfgangp added inline comments to D45549: [DWARF v5] improved support for .debug_rnglists/consumer.
May 16 2018, 10:21 AM

May 14 2018

wolfgangp updated the diff for D45549: [DWARF v5] improved support for .debug_rnglists/consumer.

Sorry for the long delay on this (I was out of the country with sketchy internet connections). I'm using the 'WithColor' convenience functions as Jonas suggested. Otherwise, I believe all review comments have been addressed.

May 14 2018, 10:53 AM

May 10 2018

wolfgangp committed rL332030: [DWARF] Fixing a bug in DWARF v5 string offsets tables where the length encoded….
[DWARF] Fixing a bug in DWARF v5 string offsets tables where the length encoded…
May 10 2018, 1:06 PM
wolfgangp closed D45922: [DWARF v5] incorrect length of string offsets tables.
May 10 2018, 1:06 PM

Apr 21 2018

wolfgangp created D45922: [DWARF v5] incorrect length of string offsets tables.
Apr 21 2018, 6:19 AM

Apr 13 2018

wolfgangp added inline comments to D45549: [DWARF v5] improved support for .debug_rnglists/consumer.
Apr 13 2018, 11:33 AM
wolfgangp updated the diff for D45549: [DWARF v5] improved support for .debug_rnglists/consumer.

Addressed review comments:

Apr 13 2018, 11:33 AM

Apr 11 2018

wolfgangp created D45549: [DWARF v5] improved support for .debug_rnglists/consumer.
Apr 11 2018, 5:57 PM

Apr 5 2018

wolfgangp committed rL329345: [DWARF v5][NFC]: Refactor DebugRnglists to prepare for the support of the….
[DWARF v5][NFC]: Refactor DebugRnglists to prepare for the support of the…
Apr 5 2018, 2:05 PM
wolfgangp closed D45307: [DWARF v5][NFC] Refactor the implementation of DebugRnglists .
Apr 5 2018, 2:05 PM
wolfgangp added inline comments to D45307: [DWARF v5][NFC] Refactor the implementation of DebugRnglists .
Apr 5 2018, 10:53 AM

Apr 4 2018

wolfgangp created D45307: [DWARF v5][NFC] Refactor the implementation of DebugRnglists .
Apr 4 2018, 6:09 PM

Mar 27 2018

wolfgangp committed rL328662: [DWARF][DWARF v5]: Adding support for dumping DW_RLE_offset_pair and….
[DWARF][DWARF v5]: Adding support for dumping DW_RLE_offset_pair and…
Mar 27 2018, 1:30 PM
wolfgangp closed D44811: DWARF] DWARF v5: .debug_rnglists: adding support for DW_RLE_offset_pair and DW_RLE_base_address.
Mar 27 2018, 1:30 PM

Mar 23 2018

wolfgangp updated the diff for D44811: DWARF] DWARF v5: .debug_rnglists: adding support for DW_RLE_offset_pair and DW_RLE_base_address.

Addressed review comment on separating DW_RLW_offset_pair and DW_RLE_start_length when dumping their contents.

Mar 23 2018, 5:23 PM
wolfgangp added inline comments to D44811: DWARF] DWARF v5: .debug_rnglists: adding support for DW_RLE_offset_pair and DW_RLE_base_address.
Mar 23 2018, 10:47 AM
wolfgangp added inline comments to D44811: DWARF] DWARF v5: .debug_rnglists: adding support for DW_RLE_offset_pair and DW_RLE_base_address.
Mar 23 2018, 10:20 AM

Mar 22 2018

wolfgangp created D44811: DWARF] DWARF v5: .debug_rnglists: adding support for DW_RLE_offset_pair and DW_RLE_base_address.
Mar 22 2018, 4:29 PM

Mar 8 2018

wolfgangp committed rL327059: [DWARF v5] Support for verbose dumping of .debug_rnglist entries.
[DWARF v5] Support for verbose dumping of .debug_rnglist entries
Mar 8 2018, 12:55 PM
wolfgangp closed D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.
Mar 8 2018, 12:55 PM

Mar 7 2018

wolfgangp added inline comments to D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.
Mar 7 2018, 6:31 PM
wolfgangp added a comment to D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Seems OK as an intermediate step - what's your end-goal/plan? I would think this would need to be merged in some way with existing range list support so that things like the symbolizer can work with DWARF 5? (see DWARFUnit::extractRangeList and DWARFDie::getAddressRanges for example)

Mar 7 2018, 6:31 PM

Mar 5 2018

wolfgangp updated the diff for D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Review comments: Aligning the closing brackets (']') of range list entries on the right, added a space to 'raw' content dumping of start/length pairs to align the contents with interval-style display.

Mar 5 2018, 6:59 PM

Feb 28 2018

wolfgangp added inline comments to D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.
Feb 28 2018, 4:31 PM
wolfgangp updated the diff for D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Addressing review comments: The dump routine(s) are using DIDumpOptions more consistently. Added a "DisplayRawContents" flag to DIDumpOptions and using it to inform DWARFAddressRange to either display a range as an interval or print its contents as is.

Feb 28 2018, 4:31 PM
wolfgangp abandoned D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..
Feb 28 2018, 10:24 AM
wolfgangp added a comment to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

Ok, souunds like the best thing to do would be to abandon this approach and start over in favor of a v5 only approach (in addition to the existing one of course).

Feb 28 2018, 10:21 AM

Feb 26 2018

wolfgangp updated the diff for D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Addressed James' review comments and gave default values to RangeListEntry's constructor parameters for better use. Added an error test for the currently unsupported DWARF64 format.

Feb 26 2018, 11:57 AM
wolfgangp added inline comments to D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.
Feb 26 2018, 11:57 AM

Feb 22 2018

wolfgangp added inline comments to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..
Feb 22 2018, 5:02 PM
wolfgangp added a comment to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

(this patch doesn't have any changes to llvm-dwp - are those missing? Did they already get reviewed/submitted? something else?)

Feb 22 2018, 4:48 PM
wolfgangp added a comment to D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Seems awkward/not ideal to compute the start/end, then reverse those computations to compute the values appropriate to print in verbose mode - probably better to keep the original values & compute the final values when needed/printing?

Also I think there's probably some similar printing logic, maybe for the debug_loc.dwo section? Maybe check that to see how the different kinds of address range specifications (start+length, start+end, base address specifier, etc) are printed there & whether there's some uniformity that can be applied/reused/improved here?

Feb 22 2018, 4:34 PM
wolfgangp updated the diff for D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.

Revised the code to take advantage of DWARFAddressRange::dump(), which simplified both non-verbose and verbose dumping. I reduced the range list entries to just raw fields of uint64_t which seems less awkward than using a union with one of the fields an AddressRange. After that there was enough commonality between verbose and non-verbose dumping that it could be handled in one function instead of two.

Feb 22 2018, 4:31 PM

Feb 21 2018

wolfgangp added inline comments to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..
Feb 21 2018, 11:07 AM
wolfgangp updated the diff for D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

Removed the test for dwo-only input since it's redundant. The 'mixed' test of dwo + dwp file covers all cases.

Feb 21 2018, 11:04 AM

Feb 15 2018

wolfgangp created D43366: [DWARF v5] Supporting verbose dumping of .dbg_rnglist entries.
Feb 15 2018, 5:57 PM
wolfgangp added a comment to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

I'm still producing the dwo files from IR, I could turn those into assembly files as well, if that's preferable.

I think it'd probably be better - could you explain what the motivation is for them being IR tests? (what's the different needs there compared to the assembly test cases?)

Feb 15 2018, 2:21 PM

Feb 14 2018

wolfgangp updated the diff for D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

Changed the test (string_offsets.test) to perform the 2 tests independently (instead of the second test depending on the result of the first). Added a hand-constructed dwp file to serve as input to test case 2. It has a mix of v5 and v4 units, type units to make sure the reference strings correctly and has the string offsets table contributions in a different order than the order in which the CUs appear in the index table.

Feb 14 2018, 6:15 PM

Feb 9 2018

wolfgangp added a comment to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

Curious combination of tests - some assembly, some IR. I think I made the existing test cases all checked in binaries.

Perhaps we could come up with a uniform approach/strategy here? (& yeah, I'd probably separate the "testing DWP input" from "testing DWP output" rather than weaving one into the other)

Feb 9 2018, 5:42 PM
wolfgangp added inline comments to D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..
Feb 9 2018, 11:03 AM
wolfgangp updated the diff for D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..

Addressed review comments, added a test for detecting DW_FORM_strp which is not supported in dwp files.

Feb 9 2018, 11:03 AM

Feb 8 2018

wolfgangp accepted D42801: [DebugInfo] Don't insert DEBUG_VALUE after terminators.

LGTM.

Feb 8 2018, 8:51 AM · debug-info

Feb 7 2018

wolfgangp added a comment to D42801: [DebugInfo] Don't insert DEBUG_VALUE after terminators.

Change LGTM with minor comment on possible readability improvement.

Feb 7 2018, 10:28 AM · debug-info

Feb 5 2018

wolfgangp created D42937: [DWARF] Make llvm-dwp handle DWARF v5 string offsets tables and indexed strings..
Feb 5 2018, 4:37 PM

Jan 30 2018

wolfgangp committed rL323804: [DWARF] Recommitting a test that was removed with r323564. Restricted to x86….
[DWARF] Recommitting a test that was removed with r323564. Restricted to x86…
Jan 30 2018, 10:43 AM

Jan 29 2018

wolfgangp committed rL323721: [DWARF] Corrected test committed in r323670 to use llc instead of llc_dwarf to….
[DWARF] Corrected test committed in r323670 to use llc instead of llc_dwarf to…
Jan 29 2018, 5:13 PM
wolfgangp committed rL323670: [DWARF] Recommitting a test reverted in r323560. Moved to x86 directory with….
[DWARF] Recommitting a test reverted in r323560. Moved to x86 directory with…
Jan 29 2018, 9:50 AM

Jan 26 2018

wolfgangp committed rL323564: [DWARF] Temporarily removing test to make buildbots happy while investigating..
[DWARF] Temporarily removing test to make buildbots happy while investigating.
Jan 26 2018, 1:26 PM
wolfgangp committed rL323560: [DWARF] Temporarily removing a test that caused an independent failure on the….
[DWARF] Temporarily removing a test that caused an independent failure on the…
Jan 26 2018, 12:51 PM
wolfgangp committed rL323546: [DWARF] Generate DWARF v5 string offsets tables along with strx* index forms..
[DWARF] Generate DWARF v5 string offsets tables along with strx* index forms.
Jan 26 2018, 10:55 AM
wolfgangp closed D42021: [DWARF] v5 implementation of string offsets tables - producer side.
Jan 26 2018, 10:55 AM

Jan 25 2018

wolfgangp added inline comments to D42021: [DWARF] v5 implementation of string offsets tables - producer side.
Jan 25 2018, 5:41 PM

Jan 24 2018

wolfgangp added a comment to D42021: [DWARF] v5 implementation of string offsets tables - producer side.

Dave, I think I addressed most of your comments, and commented on the ones I didn't. On the subject of tradeoffs wrt shared string offsets contributions, one idea would be to give each type unit (but not compile unit) its own contribution for easy removability by the linker. However, I'd like to do that or any other improvements as a follow-on patch if that's OK with you.

Jan 24 2018, 6:23 PM

Jan 17 2018

wolfgangp added inline comments to D42021: [DWARF] v5 implementation of string offsets tables - producer side.
Jan 17 2018, 6:42 PM
wolfgangp updated the diff for D42021: [DWARF] v5 implementation of string offsets tables - producer side.

Addressed most of David's review comments and commented on the ones I didn't address.

Jan 17 2018, 6:42 PM

Jan 12 2018

wolfgangp created D42021: [DWARF] v5 implementation of string offsets tables - producer side.
Jan 12 2018, 5:01 PM
wolfgangp accepted D41226: [LiveDebugValues] recognize spilled register that is killed in instruction after the spill.
Jan 12 2018, 12:00 PM

Jan 10 2018

wolfgangp committed rL322250: [DWARF][NFC] Overload AsmPrinter::emitDwarfStringOffsets() to take a….
[DWARF][NFC] Overload AsmPrinter::emitDwarfStringOffsets() to take a…
Jan 10 2018, 6:36 PM