Page MenuHomePhabricator

wolfgangp (Wolfgang Pieb)
User

Projects

User does not belong to any projects.

User Details

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

Recent Activity

Dec 11 2018

wolfgangp committed rL348896: [Debuginfo] Prevent CodeGenPrepare from dropping debuginfo references..
[Debuginfo] Prevent CodeGenPrepare from dropping debuginfo references.
Dec 11 2018, 1:17 PM
wolfgangp closed D55396: [DebugInfo] Make sure CodeGenPrepare does not drop MD references to locals..
Dec 11 2018, 1:17 PM · debug-info

Dec 10 2018

wolfgangp updated the diff for D55396: [DebugInfo] Make sure CodeGenPrepare does not drop MD references to locals..

Revised to explicitly keep track of uses in dbg value intrinsics . This is more straightforward and does not miss any edge cases as in the previous implementation. Thanks to Vedant for the suggestion.

Dec 10 2018, 5:04 PM · debug-info

Dec 6 2018

wolfgangp added inline comments to D55396: [DebugInfo] Make sure CodeGenPrepare does not drop MD references to locals..
Dec 6 2018, 5:53 PM · debug-info
wolfgangp created D55396: [DebugInfo] Make sure CodeGenPrepare does not drop MD references to locals..
Dec 6 2018, 3:58 PM · debug-info

Nov 26 2018

wolfgangp updated the diff for D41044: Implementation of -fextend-lifetimes and -fextend-this-ptr to aid with debugging of optimized code.

Rebased on r347577 with some test cases.

Nov 26 2018, 11:39 AM · debug-info
wolfgangp updated the diff for D41043: Support for an intrinsic "fake.use" (and corresponding operand) representing a use of an operand to aid debugging.

Rebased on r347573 with some test cases and bugfixes.

Nov 26 2018, 11:37 AM

Nov 15 2018

wolfgangp added inline comments to D54465: [CodeGen] Fix bugs in LiveDebugVariables when debug labels are generated..
Nov 15 2018, 2:07 PM · debug-info

Nov 2 2018

wolfgangp committed rL346061: [DWARF v5] Verifier: Add checks for DW_FORM_strx* forms..
[DWARF v5] Verifier: Add checks for DW_FORM_strx* forms.
Nov 2 2018, 5:30 PM
wolfgangp closed D54049: [DWARF v5] Verifier: Add checks for DW_FORM_strx* forms..
Nov 2 2018, 5:30 PM
wolfgangp added inline comments to D54049: [DWARF v5] Verifier: Add checks for DW_FORM_strx* forms..
Nov 2 2018, 3:07 PM
wolfgangp added inline comments to D54049: [DWARF v5] Verifier: Add checks for DW_FORM_strx* forms..
Nov 2 2018, 2:37 PM
wolfgangp created D54049: [DWARF v5] Verifier: Add checks for DW_FORM_strx* forms..
Nov 2 2018, 2:34 PM

Oct 31 2018

wolfgangp committed rL345776: [DWARF][NFC] Refactor a function to return Optional<> instead of bool.
[DWARF][NFC] Refactor a function to return Optional<> instead of bool
Oct 31 2018, 2:08 PM
wolfgangp closed D53948: [DWARF][NFC] Refactor a function to return Optional<> instead of bool.
Oct 31 2018, 2:08 PM
wolfgangp created D53948: [DWARF][NFC] Refactor a function to return Optional<> instead of bool.
Oct 31 2018, 12:00 PM

Oct 30 2018

wolfgangp committed rL345687: [DWARF] Revert r345546: Refactor range list extraction and dumping.
[DWARF] Revert r345546: Refactor range list extraction and dumping
Oct 30 2018, 6:16 PM

Oct 29 2018

wolfgangp added inline comments to D53545: [DWARF][NFC] NFC patch for reverted r342218 (refactoring rangelist handling).
Oct 29 2018, 4:50 PM
wolfgangp added inline comments to D53545: [DWARF][NFC] NFC patch for reverted r342218 (refactoring rangelist handling).
Oct 29 2018, 4:08 PM
wolfgangp committed rL345546: [DWARF][NFC] Refactor range list extraction and dumping.
[DWARF][NFC] Refactor range list extraction and dumping
Oct 29 2018, 3:19 PM
wolfgangp closed D53545: [DWARF][NFC] NFC patch for reverted r342218 (refactoring rangelist handling).
Oct 29 2018, 3:19 PM

Oct 26 2018

wolfgangp committed rL345404: [DWARF][NFC] cleanup (mostly leftovers from the implementation of string….
[DWARF][NFC] cleanup (mostly leftovers from the implementation of string…
Oct 26 2018, 10:17 AM
wolfgangp closed D53741: [DWARF][NFC] cleanup (mostly leftovers from the implementation of string offsets tables).
Oct 26 2018, 10:16 AM

Oct 25 2018

wolfgangp created D53741: [DWARF][NFC] cleanup (mostly leftovers from the implementation of string offsets tables).
Oct 25 2018, 6:19 PM

Oct 23 2018

wolfgangp added a comment to D53545: [DWARF][NFC] NFC patch for reverted r342218 (refactoring rangelist handling).

I only did a quick scan as it was reviewed before. Is there a particular area I should pay extra attention too? If not this LGTM.

Oct 23 2018, 2:29 PM

Oct 22 2018

wolfgangp created D53545: [DWARF][NFC] NFC patch for reverted r342218 (refactoring rangelist handling).
Oct 22 2018, 6:10 PM
wolfgangp added a comment to D53364: [llvm-dwarfdump] - Add the support of parsing .debug_loclists..

I just wanted to let you know that I am currently working on a patch to refactor some of the rangelists code (see: https://reviews.llvm.org/D51081) with the objective to

  1. fold the pre-v5 functionality (.debug_ranges) into DWARF5 so we can get rid of DebugRangelists.{cpp,.h}
  2. templatize more of the DWARFListTable usage so it could be also used for location lists
Oct 22 2018, 3:42 PM

Oct 19 2018

wolfgangp committed rL344807: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly..
[DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly.
Oct 19 2018, 12:25 PM
wolfgangp closed D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Oct 19 2018, 12:25 PM

Oct 12 2018

wolfgangp added inline comments to D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Oct 12 2018, 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.

Oct 12 2018, 9:44 AM

Oct 11 2018

wolfgangp added inline comments to D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Oct 11 2018, 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.
Oct 11 2018, 5:04 PM
wolfgangp created D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990..
Oct 11 2018, 11:18 AM

Oct 9 2018

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.
Oct 9 2018, 11:41 AM
wolfgangp closed D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Oct 9 2018, 11:41 AM
wolfgangp added inline comments to D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Oct 9 2018, 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.

Oct 9 2018, 10:59 AM

Oct 4 2018

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

Oct 3 2018

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.
Oct 3 2018, 3:29 PM

Sep 25 2018

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.

Sep 25 2018, 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).

Sep 25 2018, 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?

Sep 25 2018, 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).

Sep 25 2018, 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.

Sep 25 2018, 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?

Sep 25 2018, 10:44 AM

Sep 24 2018

wolfgangp created D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991).
Sep 24 2018, 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