Page MenuHomePhabricator

dstenb (David Stenberg)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 10 2016, 1:55 AM (153 w, 2 d)

Recent Activity

Wed, May 15

dstenb added inline comments to D61940: [DebugInfo] Don't always extend variable locations when the reg location is unchanging.
Wed, May 15, 5:50 AM · Restricted Project
dstenb added inline comments to D61890: [LiveDebugValues] End variable's range with multiple locations at block entry.
Wed, May 15, 2:03 AM · debug-info

Tue, May 7

dstenb added a reviewer for D61600: [DebugInfo] More precise variable range for stack locations: jmorse.

(Adding Jeremy as reviewer as he has also been working in this area recently.)

Tue, May 7, 6:22 AM · debug-info
dstenb added inline comments to D61600: [DebugInfo] More precise variable range for stack locations.
Tue, May 7, 3:43 AM · debug-info

Mon, May 6

dstenb added inline comments to D61600: [DebugInfo] More precise variable range for stack locations.
Mon, May 6, 11:56 PM · debug-info
dstenb added a comment to D58042: [LiveDebugValues] Emit parameter's entry value.

I cherry-picked the patches on top of r359425, and tried out some examples. Doing that, I encountered a case where entry values are not inserted with this patch.

Mon, May 6, 8:47 AM · debug-info

Thu, May 2

dstenb added inline comments to D60716: [DwarfDebug] Dump call site debug info into DWARF.
Thu, May 2, 5:47 AM · debug-info

Tue, Apr 30

dstenb added a comment to D58042: [LiveDebugValues] Emit parameter's entry value.

I cherry-picked the patches on top of r359425, and tried out some examples. Doing that, I encountered a case where entry values are not inserted with this patch.

Tue, Apr 30, 6:50 AM · debug-info

Mon, Apr 29

dstenb added inline comments to D60866: [DWARF] Handle DW_OP_entry_value operand.
Mon, Apr 29, 8:39 AM · debug-info
dstenb added a comment to D60961: [TargetOption] Add option for enabling param entry val tracking dbg info.

(A minor comment, which is not related to the code changes themselves, is that this should be a parent revision to D58033, and not vice-versa as it is now.)

Mon, Apr 29, 8:11 AM · debug-info

Fri, Apr 26

dstenb added inline comments to D61184: [Salvage] Change salvage debug info implementation to use new DW_OP_LLVM_convert where needed.
Fri, Apr 26, 6:31 AM · Restricted Project
dstenb added inline comments to D61184: [Salvage] Change salvage debug info implementation to use new DW_OP_LLVM_convert where needed.
Fri, Apr 26, 5:55 AM · Restricted Project

Thu, Apr 25

dstenb added inline comments to D59431: [DebugInfo] Terminate more kinds of location-list ranges at the end of basic blocks.
Thu, Apr 25, 2:47 PM · debug-info, Restricted Project
dstenb added inline comments to D59431: [DebugInfo] Terminate more kinds of location-list ranges at the end of basic blocks.
Thu, Apr 25, 12:22 PM · debug-info, Restricted Project
dstenb added inline comments to D60716: [DwarfDebug] Dump call site debug info into DWARF.
Thu, Apr 25, 1:55 AM · debug-info

Apr 10 2019

dstenb committed rGb96943b6a00d: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator (authored by dstenb).
[DebugInfo] Track multiple registers in DbgEntityHistoryCalculator
Apr 10 2019, 4:27 AM
dstenb committed rG5ffec6deef0a: [DebugInfo] Improve handling of clobbered fragments (authored by dstenb).
[DebugInfo] Improve handling of clobbered fragments
Apr 10 2019, 4:27 AM
dstenb committed rL358073: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
[DebugInfo] Track multiple registers in DbgEntityHistoryCalculator
Apr 10 2019, 4:27 AM
dstenb closed D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
Apr 10 2019, 4:27 AM · Restricted Project, debug-info
dstenb committed rL358072: [DebugInfo] Improve handling of clobbered fragments.
[DebugInfo] Improve handling of clobbered fragments
Apr 10 2019, 4:27 AM
dstenb closed D59941: [DebugInfo] Improve handling of clobbered fragments.
Apr 10 2019, 4:27 AM · Restricted Project, debug-info
dstenb committed rG6feef56d1bdb: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC (authored by dstenb).
[DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC
Apr 10 2019, 2:07 AM
dstenb committed rG3739979c2031: [DebugInfo] Make InstrRange into a class, NFC (authored by dstenb).
[DebugInfo] Make InstrRange into a class, NFC
Apr 10 2019, 2:07 AM
dstenb committed rL358060: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC.
[DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC
Apr 10 2019, 2:07 AM
dstenb closed D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC.
Apr 10 2019, 2:07 AM · Restricted Project
dstenb committed rL358059: [DebugInfo] Make InstrRange into a class, NFC.
[DebugInfo] Make InstrRange into a class, NFC
Apr 10 2019, 2:07 AM
dstenb closed D59938: [DebugInfo] Make InstrRange into a class, NFC.
Apr 10 2019, 2:07 AM · Restricted Project
dstenb committed rGfab4bdf4b93e: Add REQUIRES: asserts to test using -debug-only (authored by dstenb).
Add REQUIRES: asserts to test using -debug-only
Apr 10 2019, 1:45 AM
dstenb added a comment to D59952: [VPLAN] Minor improvement to testing and debug messages..

Hi! One of the tests was missing "REQUIRES: asserts", so it failed on a non-assert build. I took the liberty to add that to the test in r358057. I hope that was okay!

Apr 10 2019, 1:44 AM · Restricted Project
dstenb committed rL358057: Add REQUIRES: asserts to test using -debug-only.
Add REQUIRES: asserts to test using -debug-only
Apr 10 2019, 1:43 AM
dstenb added inline comments to D59938: [DebugInfo] Make InstrRange into a class, NFC.
Apr 10 2019, 1:34 AM · Restricted Project

Apr 9 2019

dstenb updated the diff for D59938: [DebugInfo] Make InstrRange into a class, NFC.

Slight adjustment: Add an isEnded() query function rather than using getEnd(). An isEnded() function is added in the follow-up patch D59941, so we can avoid unnecessary diffs in that patch by adding such a function here also.

Apr 9 2019, 8:00 AM · Restricted Project
dstenb added inline comments to D59938: [DebugInfo] Make InstrRange into a class, NFC.
Apr 9 2019, 6:58 AM · Restricted Project
dstenb updated the diff for D59938: [DebugInfo] Make InstrRange into a class, NFC.

Address review comments.

Apr 9 2019, 6:47 AM · Restricted Project
dstenb committed rG2028ae975c6a: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC (authored by dstenb).
[DebugInfo] Pass all values in DebugLocEntry's constructor, NFC
Apr 9 2019, 3:09 AM
dstenb committed rL357988: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
[DebugInfo] Pass all values in DebugLocEntry's constructor, NFC
Apr 9 2019, 3:06 AM
dstenb closed D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
Apr 9 2019, 3:06 AM · Restricted Project, debug-info
dstenb added inline comments to D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
Apr 9 2019, 2:59 AM · Restricted Project, debug-info
dstenb updated the diff for D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.

Rebase, and add Doxygen comment.

Apr 9 2019, 2:57 AM · Restricted Project, debug-info
dstenb committed rG93b497a61d20: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC (authored by dstenb).
[DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC
Apr 9 2019, 12:48 AM
dstenb committed rL357974: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC.
[DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC
Apr 9 2019, 12:48 AM
dstenb closed D59301: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC.
Apr 9 2019, 12:48 AM · Restricted Project, debug-info

Apr 8 2019

dstenb added a comment to D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

If there are remaining patches that I haven't accepted yet, please ping them.

Apr 8 2019, 9:12 AM · Restricted Project, debug-info
dstenb added a reviewer for D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC: aprantl.
Apr 8 2019, 6:43 AM · Restricted Project
dstenb added a reviewer for D59938: [DebugInfo] Make InstrRange into a class, NFC: aprantl.
Apr 8 2019, 6:42 AM · Restricted Project
dstenb added a comment to D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.

Ping.

Apr 8 2019, 6:42 AM · Restricted Project, debug-info
dstenb added a comment to D59301: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC.

Ping.

Apr 8 2019, 6:42 AM · Restricted Project, debug-info

Apr 5 2019

dstenb added a comment to D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

Thanks for the review! Just so that I don't misunderstand something; is the LGTM specifically for the changes in this patch, or does that imply the entire patch series?

Apr 5 2019, 8:55 AM · Restricted Project, debug-info

Apr 3 2019

dstenb added inline comments to D59941: [DebugInfo] Improve handling of clobbered fragments.
Apr 3 2019, 6:53 AM · Restricted Project, debug-info
dstenb added inline comments to D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
Apr 3 2019, 3:45 AM · Restricted Project, debug-info
dstenb updated the diff for D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

Address review comments.

Apr 3 2019, 3:42 AM · Restricted Project, debug-info

Apr 2 2019

dstenb updated the diff for D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

Rebase after update of D59941.

Apr 2 2019, 7:36 AM · Restricted Project, debug-info
dstenb added inline comments to D59941: [DebugInfo] Improve handling of clobbered fragments.
Apr 2 2019, 7:36 AM · Restricted Project, debug-info
dstenb updated the diff for D59941: [DebugInfo] Improve handling of clobbered fragments.

Doxygenize comments.

Apr 2 2019, 7:31 AM · Restricted Project, debug-info
dstenb updated the diff for D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

Address comments.

Apr 2 2019, 3:49 AM · Restricted Project, debug-info
dstenb added inline comments to D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
Apr 2 2019, 3:49 AM · Restricted Project, debug-info
dstenb added a comment to D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.

Is there any performance hit?

Apr 2 2019, 2:03 AM · Restricted Project, debug-info
dstenb added a comment to D59941: [DebugInfo] Improve handling of clobbered fragments.

What's the performance impact of this change? Specifically, I'd be interested in the wall clock time difference for building an RelWithDebInfo+asan build of clang.

Apr 2 2019, 1:08 AM · Restricted Project, debug-info

Mar 28 2019

dstenb added inline comments to D59431: [DebugInfo] Terminate more kinds of location-list ranges at the end of basic blocks.
Mar 28 2019, 9:16 AM · debug-info, Restricted Project
dstenb added a parent revision for D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator: D59941: [DebugInfo] Improve handling of clobbered fragments.
Mar 28 2019, 8:59 AM · Restricted Project, debug-info
dstenb added a child revision for D59941: [DebugInfo] Improve handling of clobbered fragments: D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
Mar 28 2019, 8:59 AM · Restricted Project, debug-info
dstenb created D59942: [DebugInfo] Track multiple registers in DbgEntityHistoryCalculator.
Mar 28 2019, 8:59 AM · Restricted Project, debug-info
dstenb added a comment to D59941: [DebugInfo] Improve handling of clobbered fragments.

This patch depends on four preceding refactoring commits. I avoided adding reviewers to two of them (I should perhaps have done the same for the other two) to avoid spamming all of you (who probably deals with enough revisions as it is), but they are up for review.

Mar 28 2019, 8:58 AM · Restricted Project, debug-info
dstenb added a child revision for D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC: D59941: [DebugInfo] Improve handling of clobbered fragments.
Mar 28 2019, 8:53 AM · Restricted Project
dstenb added a parent revision for D59941: [DebugInfo] Improve handling of clobbered fragments: D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC.
Mar 28 2019, 8:53 AM · Restricted Project, debug-info
dstenb created D59941: [DebugInfo] Improve handling of clobbered fragments.
Mar 28 2019, 8:53 AM · Restricted Project, debug-info
dstenb added a child revision for D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC: D59938: [DebugInfo] Make InstrRange into a class, NFC.
Mar 28 2019, 8:53 AM · Restricted Project, debug-info
dstenb added a parent revision for D59938: [DebugInfo] Make InstrRange into a class, NFC: D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
Mar 28 2019, 8:53 AM · Restricted Project
dstenb added a parent revision for D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC: D59938: [DebugInfo] Make InstrRange into a class, NFC.
Mar 28 2019, 8:42 AM · Restricted Project
dstenb added a child revision for D59938: [DebugInfo] Make InstrRange into a class, NFC: D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC.
Mar 28 2019, 8:42 AM · Restricted Project
dstenb created D59939: [DebugInfo] Rename DbgValueHistoryMap::{InstrRange -> Entry}, NFC.
Mar 28 2019, 8:42 AM · Restricted Project
dstenb created D59938: [DebugInfo] Make InstrRange into a class, NFC.
Mar 28 2019, 8:39 AM · Restricted Project
dstenb added a comment to D59790: [DebugInfo][Docs] Document how variable location metadata is transformed through target codegen.

I have not read through everything yet, but thanks for documenting this!

Mar 28 2019, 8:34 AM · Restricted Project

Mar 18 2019

dstenb committed rG8a2e4af7e750: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values (authored by dstenb).
[DebugInfo] Ignore bitcasts when lowering stack arg dbg.values
Mar 18 2019, 4:28 AM
dstenb committed rL356363: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
[DebugInfo] Ignore bitcasts when lowering stack arg dbg.values
Mar 18 2019, 4:27 AM
dstenb closed D58831: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
Mar 18 2019, 4:26 AM · Restricted Project, debug-info

Mar 13 2019

dstenb updated the diff for D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.

Move lambda expression back into assert() to avoid -Wunused-variable warning in non-assert builds.

Mar 13 2019, 3:41 PM · Restricted Project, debug-info
Herald added a project to D57145: [DebugInfo] Remove redundant initialization of DbgVariable's MInsn, NFC: Restricted Project.

Ping.

Mar 13 2019, 9:12 AM · Restricted Project, debug-info
dstenb added a parent revision for D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC: D59301: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC.
Mar 13 2019, 9:11 AM · Restricted Project, debug-info
dstenb added a child revision for D59301: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC: D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
Mar 13 2019, 9:11 AM · Restricted Project, debug-info
dstenb created D59303: [DebugInfo] Pass all values in DebugLocEntry's constructor, NFC.
Mar 13 2019, 9:10 AM · Restricted Project, debug-info
dstenb created D59301: [DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC.
Mar 13 2019, 9:07 AM · Restricted Project, debug-info
dstenb added inline comments to D58831: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
Mar 13 2019, 9:02 AM · Restricted Project, debug-info

Mar 7 2019

dstenb committed rG27ed855a6e1f: [analyzer] Handle comparison between non-default AS symbol and constant (authored by dstenb).
[analyzer] Handle comparison between non-default AS symbol and constant
Mar 7 2019, 5:02 AM
dstenb committed rL355592: [analyzer] Handle comparison between non-default AS symbol and constant.
[analyzer] Handle comparison between non-default AS symbol and constant
Mar 7 2019, 5:02 AM
dstenb committed rC355592: [analyzer] Handle comparison between non-default AS symbol and constant.
[analyzer] Handle comparison between non-default AS symbol and constant
Mar 7 2019, 5:02 AM
dstenb closed D58665: [analyzer] Handle comparison between non-default AS symbol and constant.
Mar 7 2019, 5:02 AM · Restricted Project
dstenb added a comment to D58665: [analyzer] Handle comparison between non-default AS symbol and constant.

Thanks for the review! I'll submit this shortly then.

Mar 7 2019, 12:29 AM · Restricted Project

Mar 6 2019

dstenb added inline comments to D58831: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
Mar 6 2019, 12:47 AM · Restricted Project, debug-info

Mar 5 2019

dstenb added inline comments to D58831: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
Mar 5 2019, 7:43 AM · Restricted Project, debug-info

Mar 1 2019

dstenb created D58831: [DebugInfo] Ignore bitcasts when lowering stack arg dbg.values.
Mar 1 2019, 7:29 AM · Restricted Project, debug-info
dstenb added a comment to D58726: [DebugInfo][Docs] Explicitly document how dbg.value intrinsics are interpreted in optimized code.

Some nitpicks.

Mar 1 2019, 4:44 AM · Restricted Project

Feb 27 2019

dstenb updated the diff for D58665: [analyzer] Handle comparison between non-default AS symbol and constant.

Address comments.

Feb 27 2019, 7:53 AM · Restricted Project

Feb 26 2019

dstenb created D58665: [analyzer] Handle comparison between non-default AS symbol and constant.
Feb 26 2019, 1:25 AM · Restricted Project

Feb 18 2019

dstenb added inline comments to D58042: [LiveDebugValues] Emit parameter's entry value.
Feb 18 2019, 5:08 AM · debug-info

Feb 14 2019

dstenb abandoned D57587: NOT FOR COMMIT Print debug values in beginFunction().
Feb 14 2019, 3:51 AM

Feb 13 2019

dstenb added inline comments to D58042: [LiveDebugValues] Emit parameter's entry value.
Feb 13 2019, 1:19 PM · debug-info
dstenb added inline comments to D58044: [DwarfDebug] Dump call site debug info into DWARF.
Feb 13 2019, 8:27 AM · debug-info
dstenb added inline comments to D58041: [Backend] DBG_CALLSITE & DBG_CALLSITEPARAM instr handling.
Feb 13 2019, 3:08 AM · debug-info
dstenb added inline comments to D58041: [Backend] DBG_CALLSITE & DBG_CALLSITEPARAM instr handling.
Feb 13 2019, 3:08 AM · debug-info