JDevlieghere added a comment.

I believe it's generally considered bad practice to install signal handlers in 
a library. This is why the initialization currently happens in the driver. If 
you look at other tools in LLVM you'll notice they do the same thing.

Can we address this issue by (1) making sure we don't crash in lldbassert when 
calling `PrintStackTrace` without having initialized the signal handlers and 
(2) providing the ability to install the signal handlers through the SB API (if 
that doesn't already exist) if the user of libLLDB wants this functionality?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119009/new/

https://reviews.llvm.org/D119009

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to