This is an archive of the discontinued LLVM Phabricator instance.

Change CFA to be a LoadAddress instead of HostAddress
ClosedPublic

Authored by ribrdb on Oct 10 2014, 4:31 PM.

Details

Reviewers
jasonmolenda
Summary

The computed CFA is an address in the target process, not lldb. So printing variables which reference the CFA results in a crash.
See http://llvm.org/bugs/show_bug.cgi?id=21118

Diff Detail

Event Timeline

ribrdb updated this revision to Diff 14757.Oct 10 2014, 4:31 PM
ribrdb retitled this revision from to Change CFA to be a LoadAddress instead of HostAddress.
ribrdb updated this object.
ribrdb edited the test plan for this revision. (Show Details)
ribrdb added a subscriber: Unknown Object (MLST).

Not sure if this worked since I wasn't subscribed to the mailing list.

emaste added a subscriber: emaste.Oct 15 2014, 12:53 PM
jasonmolenda accepted this revision.Oct 15 2014, 8:06 PM
jasonmolenda added a reviewer: jasonmolenda.

Landed in r219896

This revision is now accepted and ready to land.Oct 15 2014, 8:06 PM
jasonmolenda closed this revision.Oct 15 2014, 8:07 PM