This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Don't install the pretty backtrace handlers twice.
ClosedPublic

Authored by JDevlieghere on Nov 13 2019, 5:28 PM.

Details

Summary

I noticed that currently we are printing LLVM's pretty stack trace
twice. The reason is that we're calling PrintStackTraceOnErrorSignal in
addition to InitLLVM, which besides some other useful things, also
register LLVM's pretty stack trace handler.

Diff Detail

Event Timeline

JDevlieghere created this revision.Nov 13 2019, 5:28 PM
Herald added a project: Restricted Project. · View Herald Transcript

Fold ToolName into printHelp

vsk accepted this revision.Nov 13 2019, 5:33 PM

Looks good to me!

This revision is now accepted and ready to land.Nov 13 2019, 5:33 PM
This revision was automatically updated to reflect the committed changes.

Cool. I saw that we print stack trace twice, but I thought that was because of the whole liblldb+lldb signal double installation. Thanks for fixing this.