[Lldb-commits] [PATCH] D120425: [lldb/host] Remove monitor_signals argument from process monitoring functions

2022-02-24 Thread Nico Weber via Phabricator via lldb-commits
thakis added a comment. Breaks Mac build: http://45.33.8.238/macm1/28811/step_4.txt Ptal. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120425/new/ https://reviews.llvm.org/D120425 ___ lldb-commits

[Lldb-commits] [PATCH] D120425: [lldb/host] Remove monitor_signals argument from process monitoring functions

2022-02-24 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG12c9c4a88537: [lldb/host] Remove monitor_signals argument from process monitoring functions (authored by labath). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[Lldb-commits] [PATCH] D120425: [lldb/host] Remove monitor_signals argument from process monitoring functions

2022-02-23 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. Nice Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120425/new/ https://reviews.llvm.org/D120425

[Lldb-commits] [PATCH] D120425: [lldb/host] Remove monitor_signals argument from process monitoring functions

2022-02-23 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added a reviewer: JDevlieghere. Herald added a subscriber: krytarowski. labath requested review of this revision. Herald added a project: LLDB. All current callers set the argument to false. monitor_signals=true used to be used in the Process plugins (which