[Lldb-commits] [PATCH] D74657: [lldb/Target] Add process crash-info command

2020-02-17 Thread Pavel Labath via Phabricator via lldb-commits
labath requested changes to this revision. labath added a comment. This revision now requires changes to proceed. All of this code you're adding to Process.cpp is extremely specific to one platform's notion of "crash info". This should be factored out so that macos-specific details live inside s

[Lldb-commits] [PATCH] D74657: [lldb/Target] Add process crash-info command

2020-02-15 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor added a comment. I have some comments (beside my usual documentation spam). But otherwise looks good to me. Comment at: lldb/bindings/interface/SBProcess.i:196 +SBStructuredData +GetCrashInfo (); + documentation. Comment at

[Lldb-commits] [PATCH] D74657: [lldb/Target] Add process crash-info command

2020-02-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib updated this revision to Diff 244793. mib added a comment. Run clang-format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74657/new/ https://reviews.llvm.org/D74657 Files: lldb/bindings/interface/SBProcess.i lldb/include/lldb/API/SBProcess

[Lldb-commits] [PATCH] D74657: [lldb/Target] Add process crash-info command

2020-02-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib created this revision. mib added reviewers: JDevlieghere, friss, jingham. mib added a project: LLDB. Herald added a subscriber: lldb-commits. Currently, in macOS, when a process crashes, lldb halts inside the implementation disassembly without yielding any useful information. The only way to