- User Since
- Mar 5 2019, 4:12 AM (47 w, 10 h)
Thu, Jan 23
Fix spelling error.
Add test, address nit comment. Note that some of the current test checks should pass even without this patch if commit 636c93 is not reverted, as some of the redundant debug info would be dropped.
Mon, Jan 20
Wed, Jan 15
Dec 19 2019
Merged in commit: llvmorg-10-init-12272-g89d19d60adb9
Dec 18 2019
Dec 17 2019
Format, addressing review comments
@djtodoro Does this latest change look good to you?
Dec 13 2019
Use shared functionality for zero/sign extension ops, fix tests
Dec 12 2019
Rebased and updated for latest master, fixed clang version and unused attributes in tests.
Dec 10 2019
Dec 6 2019
Dec 4 2019
Use Undef int1 when undef-ing a DbgVariable without a type.
Maintain order of debug intrinsics in empty block elimination, fix test
Dec 3 2019
Merged in commit: llvmorg-10-init-11043-g269a9afe25cb
Rebased on master, rename getVar -> getVariable
Dec 2 2019
Use another lambda to simplify skipping of dbg info, fix incorrect type usage in setDbgVariableUndef
Remove redundant assignment on test variable construction
Add unit test, allow nullptr for DIExpression in DebugVariable ctor
Nov 28 2019
Use NoneType() to call correct DebugVariable constructor
Clean up code, fix comments.
Nov 27 2019
Rebased off of merged parent, trimmed unnecessary/confusing test metadata.
Nov 26 2019
Nov 22 2019
Fix formatting, include whole files in patch (mistakenly removed in prior revision).
Nov 21 2019
Nov 20 2019
Show correct changeset, preserve operands for dbg.declares
Nov 19 2019
Updating according to review comments; the code in Local.cpp needs further work, but is close to its likely final state. This version also applies to master, so diffing the latest revision with the previous will give a lot of noise.
Nov 15 2019
Nov 14 2019
Nov 5 2019
Sep 9 2019
This one has been merged in; it probably didn't get picked up because this review doesn't include the other changes that were reapplied in the patch.
Aug 20 2019
The regression issue is documented in the bug above: https://bugs.llvm.org/show_bug.cgi?id=41931
The main issue can be summarised briefly as "splitting a fragment DIExpression across multiple registers is currently broken", which is what I've submitted a fix for (see comments in SelectionDAGBuilder.cpp); the test for the new behaviour is in fragmented-args-multiple-regs.ll
Aug 16 2019
Reopening, as the merged patch caused a regression issue and needed to be reverted; this update should fix said regression issue.
Jul 26 2019
Looks like a sensible straightforward change.
May 16 2019
May 15 2019
May 14 2019
May 3 2019
Remove incorrectly included lines.
May 2 2019
Updated to use convert OP correctly as suggested by markus.
Apr 26 2019
As per the previous comment, ZExt is equivalent to a Noop on the DWARF expression stack, so can be caught by the same early exit.
Apr 15 2019
Apr 8 2019
Fixes an Object test that I missed in the last revision.
Apr 5 2019
Mar 29 2019
Remove unnecessary/unrelated changes.
Minor code cleanups.
Mar 28 2019
Mar 15 2019
Update comment to give more context to the use of a Line 0 debug location.
Mar 11 2019
Mar 8 2019
Added the test case for the second change, and reduced both it and the original test case as much as possible.