[Lldb-commits] [PATCH] D49377: Move pretty stack trace printer into driver.

2018-07-17 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL337261: Move pretty stack trace printer into driver. (authored by JDevlieghere, committed by ). Changed prior to commit: https://reviews.llvm.org/D49377?vs=155684&id=155834#toc Repository: rL LLVM h

[Lldb-commits] [PATCH] D49377: Move pretty stack trace printer into driver.

2018-07-16 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere updated this revision to Diff 155684. JDevlieghere added a comment. Herald added a subscriber: ki.stfu. I've added it to the tools that made sense to me. Let me know if I missed something obvious. https://reviews.llvm.org/D49377 Files: source/Initialization/SystemInitializerComm

[Lldb-commits] [PATCH] D49377: Move pretty stack trace printer into driver.

2018-07-16 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. I think this makes perfect sense. Could you also add the same thing to the other binaries in the tools subfolder? https://reviews.llvm.org/D49377 __

[Lldb-commits] [PATCH] D49377: Move pretty stack trace printer into driver.

2018-07-16 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: jingham, labath, zturner. Herald added a reviewer: jfb. We used to have a pretty stack trace printer in SystemInitializerCommon. This was disabled on Apple because we didn't want the library to be setting signal handlers, as this