do not crash process by default when ExceptionTracer fails to reflect over cxxabi
authorBenjamin Reesman <benr@fb.com>
Thu, 20 Jul 2017 07:08:50 +0000 (00:08 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Thu, 20 Jul 2017 07:24:22 +0000 (00:24 -0700)
commit05a686a79a3babca628fd3be34c86a6140031970
treed96f08be7f7339fcf39a47fd7aced88353416b52
parente1fb97e7c0bf8176ff9b72493ecf14d7781559b0
do not crash process by default when ExceptionTracer fails to reflect over cxxabi

Summary: It is possible in certain cirucmstances for the cxxabi reflection code in `ExceptionTracer` to fail to cope with exceptions that we've seen in production and it currently aborts the process in that case. This diff switches this to `DFATAL`/`DCHECK`.

Reviewed By: yfeldblum

Differential Revision: D5431445

fbshipit-source-id: c3d68372c2fadbb518f78fe99e817db611953d22
folly/experimental/exception_tracer/ExceptionTracer.cpp