This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Fix help text for "platform settings"
ClosedPublic

Authored by DavidSpickett on Oct 28 2022, 2:18 AM.

Details

Summary

This claims to take a platform name argument but doesn't.

That was probably the intent in fbb7634934d40548b650574a2f2a85ab41527674
but it has only ever worked with the current platform.

Diff Detail

Event Timeline

DavidSpickett created this revision.Oct 28 2022, 2:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 28 2022, 2:18 AM
DavidSpickett requested review of this revision.Oct 28 2022, 2:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 28 2022, 2:18 AM

Alternatively, I could look at adding that option. Not sure off the top of my head if configuring an inactive platform would even work.

clayborg accepted this revision.Jan 13 2023, 4:14 PM

Sorry for the delay!

This revision is now accepted and ready to land.Jan 13 2023, 4:14 PM
This revision was automatically updated to reflect the committed changes.