[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 20 11:52:02 PDT 2019
rnk added a subscriber: MatzeB.
rnk added a comment.
This has been proposed before by @MatzeB:
https://reviews.llvm.org/D33960
At the time, there was an issue that some people had been "fixing" fuzzer bugs by replacing asserts with `if (cond) report_fatal_error(...);`. Moving to abort will reopen all of those fuzzer bugs. However, I think we have consensus that that is the desired behavior: every call to report_fatal_error is essentially a bug,. Code should be using the error handlers on MCContext and LLVMContext instead and recovering.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67847/new/
https://reviews.llvm.org/D67847
More information about the llvm-commits
mailing list