Page MenuHomePhabricator

[WIP][DebugInfo] LiveDebugValues should allow different variable fragments to have different locations

Authored by jmorse on May 24 2019, 6:11 AM.



[Work in progress, not really up for review]

Here's a patch that makes LiveDebugValues recognise different variable fragments as having different locations, allowing it to propagate fragments independently (see: the regression test).

There's an obvious flaw with this : *overlapping* fragments will be recognised as being different fragments, and will get propagated, leading to unsound locations being extended. What's not clear to me is whether this is best dealt with at a different layer, which I'll write up in the bug report at

Diff Detail

Event Timeline

jmorse created this revision.May 24 2019, 6:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 24 2019, 6:11 AM
jmorse abandoned this revision.Jun 5 2019, 7:42 AM

Superceeded by D62904, I felt it cleaner to upload a new review req.