This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] dotest.py set selected_platform on remote connection
ClosedPublic

Authored by omjavaid on Jun 28 2021, 1:23 PM.

Details

Summary

This patch fixes a bug in dotest.py where lldb.selected_platform was
being set to host platform even after a successful connection to a
remote platform via platform url. This patch fixes this behavior and
sets selected_platform to remote_platform after a successful connection.

This patch also removes target_platform variable from run_suite.

Diff Detail

Event Timeline

omjavaid requested review of this revision.Jun 28 2021, 1:23 PM
omjavaid created this revision.
JDevlieghere accepted this revision.Jun 28 2021, 1:37 PM

Thanks! LGTM

This revision is now accepted and ready to land.Jun 28 2021, 1:37 PM
This revision was landed with ongoing or failed builds.Jun 28 2021, 2:37 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJun 28 2021, 2:37 PM