[Lldb-commits] [PATCH] D37651: Fix for bug 34532 - A few rough corners related to post-mortem debugging (core/minidump)

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Sep 8 18:11:41 PDT 2017


jingham added a comment.

lldb is a library used in other programs (Xcode, VSCode etc) and it can support many simultaneous debug sessions and each debug session can support many simultaneous targets.  Unless this failure is going to make all the other debug sessions fail, and the other target sessions fail and cause the app which has loaded us to fail, we have no business crashing.

I know there is debate about this one side and another but for lldb this is a settled issue.  Unless you really are in a state where the world is about to come crashing down around you, you can't raise a fatal error in lldb.  And in this case, the world is only very minorly strange, so it is certainly not appropriate.


https://reviews.llvm.org/D37651





More information about the lldb-commits mailing list