[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-04 Thread walter erquinigo via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGdb5f745a2bfc: [lldb] Make some functions useful to REPLs public (authored by wallace). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149717/new/

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-04 Thread Alex Langford via Phabricator via lldb-commits
bulbazord accepted this revision. bulbazord added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149717/new/ https://reviews.llvm.org/D149717

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-04 Thread walter erquinigo via Phabricator via lldb-commits
wallace updated this revision to Diff 519542. wallace added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149717/new/ https://reviews.llvm.org/D149717 Files: lldb/include/lldb/Core/Debugger.h Index:

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-04 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added inline comments. Comment at: lldb/include/lldb/Core/Debugger.h:505 + /// that directly use the \a lldb_private namespace and want to use the + /// default LLDB event handler thread instead of defining their own. + bool StartEventHandlerThread();

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-03 Thread walter erquinigo via Phabricator via lldb-commits
wallace updated this revision to Diff 519141. wallace added a comment. address comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149717/new/ https://reviews.llvm.org/D149717 Files: lldb/include/lldb/Core/Debugger.h Index:

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-03 Thread walter erquinigo via Phabricator via lldb-commits
wallace added inline comments. Comment at: lldb/include/lldb/Core/Debugger.h:503-504 + /// Manually start the global event handler thread. It should be used by + /// programs that use LLDB as a library. + bool StartEventHandlerThread(); bulbazord wrote: >

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-03 Thread Alex Langford via Phabricator via lldb-commits
bulbazord added a reviewer: JDevlieghere. bulbazord added inline comments. Comment at: lldb/include/lldb/Core/Debugger.h:503-504 + /// Manually start the global event handler thread. It should be used by + /// programs that use LLDB as a library. + bool

[Lldb-commits] [PATCH] D149717: [lldb] Make some functions useful to REPLs public

2023-05-02 Thread walter erquinigo via Phabricator via lldb-commits
wallace created this revision. Herald added a project: All. wallace requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. `StartEventHandlerThread` and `StopEventHandlerThread` are available to the SwiftREPL even though they are protected