diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2018-06-15 16:45:19 +0000 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2018-06-15 16:45:19 +0000 |
commit | 4adf24502ec8c993e3da2fe0ad2f4921e5a76bb9 (patch) | |
tree | d67950f68ed4bb6455dcaf9a9041a235d274fadd /clang-tools-extra/clang-query | |
parent | a6edca72ba6962455ab427759b9380555db0d211 (diff) | |
download | bcm5719-llvm-4adf24502ec8c993e3da2fe0ad2f4921e5a76bb9.tar.gz bcm5719-llvm-4adf24502ec8c993e3da2fe0ad2f4921e5a76bb9.zip |
[scudo] Add verbose failures in place of CHECK(0)
Summary:
The current `FailureHandler` mechanism was fairly opaque with regard to the
failure reason due to using `CHECK(0)`. Scudo is a bit different from the other
Sanitizers as it prefers to avoid spurious processing in its failure path. So
we just `dieWithMessage` using a somewhat explicit string.
Adapted the tests for the new strings.
While this takes care of the `OnBadRequest` & `OnOOM` failures, the next step
is probably to migrate the other Scudo failures in the same failes (header
corruption, invalid state and so on).
Reviewers: alekseyshl
Reviewed By: alekseyshl
Subscribers: filcab, mgorny, delcypher, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D48199
llvm-svn: 334843
Diffstat (limited to 'clang-tools-extra/clang-query')
0 files changed, 0 insertions, 0 deletions