Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

Add backtraces for debugging. #80

Merged
merged 1 commit into from
Jul 6, 2022
Merged

Add backtraces for debugging. #80

merged 1 commit into from
Jul 6, 2022

Conversation

varungandhi-src
Copy link
Contributor

I've directly copied the header from backward-cpp for now,
as it is also MIT licensed, and this seemed simpler than having
to futz more with CMake and/or submodules.

This fixes both issues:

  • If there is a crash in the driver process, we print a stack trace.
  • If there is a crash in the individual threads, we still print a stack trace.

I'm not 100% sure if this will work with OOM, but there doesn't seem to be any
harm in trying.

Fixes #73.

Test plan

Manually tested. See instructions in

I've directly copied the code from backward-cpp for now,
as it is also MIT licensed, and this seemed
simpler than having to futz more with CMake and/or submodules.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check stack traces and use backward-cpp if it isn't working
1 participant