Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Fix lldb-mi test data_read_memory_bytes_global

Authored by ted on Sep 6 2017, 1:46 PM.



Test was skipped because -data-evaluate-expression was thought
to not work on globals. This is not the case - the issue was clang
removes debug info for globals in cpp files that are not used.

Add a reference to the globals in question, and fix memory patter in
test to match memory pattern in testcase.

Event Timeline

ted created this revision.Sep 6 2017, 1:46 PM
ted added a comment.Sep 6 2017, 1:48 PM

Tested with clang 3.8 on top-of-tree lldb-mi on Ubuntu 14.

ki.stfu accepted this revision.Sep 7 2017, 6:20 AM


This revision is now accepted and ready to land.Sep 7 2017, 6:20 AM
ted closed this revision.Sep 7 2017, 9:25 AM