wolfgangp (Wolfgang Pieb)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 17 2014, 5:52 PM (213 w, 1 d)

Recent Activity

Fri, Oct 12

wolfgangp added inline comments to D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Fri, Oct 12, 9:44 AM
wolfgangp updated the diff for D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..

Made the location list section pointer and the stringref that describes its data a union rather than a pair.

Fri, Oct 12, 9:44 AM

Thu, Oct 11

wolfgangp added inline comments to D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Thu, Oct 11, 5:04 PM
wolfgangp updated the diff for D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..

Addressed review comments:

  • Keeping track of locsection data separately so we can adjust it when handling dwp files.
  • removing a cleanup that can be applied later.
Thu, Oct 11, 5:04 PM
wolfgangp created D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Thu, Oct 11, 11:18 AM

Tue, Oct 9

wolfgangp committed rL344068: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section. Fixes PR38991..
[DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section. Fixes PR38991.
Tue, Oct 9, 11:41 AM
wolfgangp closed D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Tue, Oct 9, 11:41 AM
wolfgangp added inline comments to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Tue, Oct 9, 10:59 AM
wolfgangp updated the diff for D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

Eliminated one of the test cases due to redundancy and adjusted a comment in the other one.

Tue, Oct 9, 10:59 AM

Thu, Oct 4

wolfgangp added inline comments to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Thu, Oct 4, 12:07 PM

Wed, Oct 3

wolfgangp updated the diff for D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
  • Removed the condition on the existence of compile units.
  • Added a test to check that llvm-dwarfdump can dump the .debug_loc.dwo section even without compile units present.
Wed, Oct 3, 3:29 PM

Tue, Sep 25

wolfgangp updated the diff for D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

Hardcoding address size to 4 as a workaround with a FIXME comment. DWARFExpression is unaware of DWO and requires it at the moment.

Tue, Sep 25, 4:53 PM
wolfgangp added a comment to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

If it's hard-coded to anything other than 4 or 8 the constructor for DWARFExpression would assert. If it's hard-coded to 4 it would work just fine (except for the DW_OP_call_ref example with an 8-byte operand mentioned above, and that would probably just report an obscure error and move on to the next list).

Tue, Sep 25, 3:28 PM
wolfgangp added a comment to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

Should the address size be required/used for parsing the debug_loc.dwo
section? I would think there shouldn't be any addresses, or otherwise
address-size-dependent there?

Yes, but currently DWARFExpression is oblivious to DWO and insists on having a valid Address Size. This would have to be fixed before we can omit the address size.

Is that sufficiently expensive to be worth doing this workaround in the interim?

The question is whether to bring it up to DWARF5 level right away or to just have it deal with DWO with what's currently supported. The former would certainly be more expensive.

Not sure I follow what you mean by that - what it has to do with DWARF5 or not. The pre-DWARF5 version of split-DWARF wouldn't've used addresses in the .dwo file - the addresses aren't known at that time, I think?

Tue, Sep 25, 3:22 PM
wolfgangp added a comment to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

One reason why we may still want addrsize is so we can skip the correct amount of bytes during parsing if we do find a DW_OP_addr in a DWO (and report an error).

Tue, Sep 25, 2:47 PM
wolfgangp added a comment to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

Should the address size be required/used for parsing the debug_loc.dwo
section? I would think there shouldn't be any addresses, or otherwise
address-size-dependent there?

Yes, but currently DWARFExpression is oblivious to DWO and insists on having a valid Address Size. This would have to be fixed before we can omit the address size.

Is that sufficiently expensive to be worth doing this workaround in the interim?

The question is whether to bring it up to DWARF5 level right away or to just have it deal with DWO with what's currently supported. The former would certainly be more expensive.

Tue, Sep 25, 2:37 PM
wolfgangp added a comment to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).

Should the address size be required/used for parsing the debug_loc.dwo
section? I would think there shouldn't be any addresses, or otherwise
address-size-dependent there?

Tue, Sep 25, 10:44 AM

Mon, Sep 24

wolfgangp created D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Mon, Sep 24, 4:21 PM

Sep 14 2018

wolfgangp committed rL342218: [DWARF] reposting r342048, which was reverted in r342056 due to buildbot .
[DWARF] reposting r342048, which was reverted in r342056 due to buildbot
Sep 14 2018, 2:15 AM

Sep 12 2018

wolfgangp committed rL342056: Reverting r342048, which caused UBSan failures in dsymutil..
Reverting r342048, which caused UBSan failures in dsymutil.
Sep 12 2018, 7:41 AM
wolfgangp committed rL342048: [DWARF] Refactoring range list dumping to fold DWARF v4 functionality into v5….
[DWARF] Refactoring range list dumping to fold DWARF v4 functionality into v5…
Sep 12 2018, 5:02 AM
wolfgangp closed D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Sep 12 2018, 5:02 AM
wolfgangp added a comment to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..

@wolfgangp do you need any help with this revision before you commit it?

Sep 12 2018, 2:52 AM

Sep 7 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Sep 7 2018, 5:55 PM
wolfgangp updated the diff for D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..

Added some checks to cu-ranges.ll to capture the intent of the test better.

Sep 7 2018, 5:55 PM

Aug 30 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 30 2018, 4:59 PM
wolfgangp updated the diff for D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..

When we find an error during table extraction, ListTableBase::extract is expected to set Offset to either 0 if the error is not recoverable, or to past the end of the table if it is. This removes the need to access the extracted length in the caller in the error case and simplifies recovery.

Aug 30 2018, 4:53 PM

Aug 29 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 29 2018, 6:09 PM

Aug 28 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 28 2018, 5:45 PM
wolfgangp updated the diff for D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..

Addressed review comments:

  • refactored getMaxVersion() in DWARFUnit.h to take an optional default argument.
  • Removed some code that is a noop for this patch
  • Extracting the initial length field of a list table to a local instead of straight into the header data. This aids error recovery when we find an unsupported DWARF64 length field.
  • added a missing comment
  • corrected one coding guides violation.
Aug 28 2018, 5:44 PM

Aug 27 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 27 2018, 10:13 AM

Aug 23 2018

wolfgangp added inline comments to D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 23 2018, 5:46 PM
wolfgangp updated the diff for D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..

Addressed David's review comments:

Aug 23 2018, 5:46 PM

Aug 21 2018

wolfgangp created D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC)..
Aug 21 2018, 6:17 PM

Jul 31 2018

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.
Jul 31 2018, 1:57 PM
wolfgangp closed D50089: [DWARF v4] Suppressing the __debug_ranges section when there are no ranges.
Jul 31 2018, 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.
Jul 31 2018, 12:38 PM
wolfgangp created D50089: [DWARF v4] Suppressing the __debug_ranges section when there are no ranges.
Jul 31 2018, 11:09 AM

Jul 30 2018

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

Jul 26 2018

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…
Jul 26 2018, 3:49 PM

Jul 25 2018

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…
Jul 25 2018, 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).
Jul 25 2018, 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…
Jul 25 2018, 4:03 PM
wolfgangp closed D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Jul 25 2018, 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).
Jul 25 2018, 11:56 AM

Jul 24 2018

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.
Jul 24 2018, 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?

Jul 24 2018, 3:34 PM

Jul 23 2018

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.
Jul 23 2018, 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.

Jul 23 2018, 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…
Jul 23 2018, 3:37 PM
wolfgangp closed D49500: [DWARF v5] Refactor range lists dumping (NFC except for spelling changes in dump output)..
Jul 23 2018, 3:37 PM

Jul 20 2018

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
Jul 20 2018, 11:53 AM

Jul 19 2018

wolfgangp added inline comments to D49522: [DWARF v5] Don't emit duplicate DW_AT_rnglists_base and address review comments from previous review.
Jul 19 2018, 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.

Jul 19 2018, 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.
Jul 19 2018, 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.

Jul 19 2018, 11:35 AM

Jul 18 2018

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

Jul 16 2018

wolfgangp added inline comments to D49214: [DWARF v5] emit DWARF v5 range lists (no support for fission yet).
Jul 16 2018, 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