- User Since
- Mar 5 2019, 4:12 AM (28 w, 9 h)
Mon, Sep 9
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.
Tue, Aug 20
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.