diff options
| author | Kostya Kortchinsky <kostyak@google.com> | 2018-09-20 14:50:06 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2018-09-20 14:50:06 +0000 |
| commit | caa9619bcef4e67f5c2d4603b1d67c4cca6cacd8 (patch) | |
| tree | 255de32303966c0ae6d0d07bc057d9bc0a2a923b /llvm | |
| parent | 0ff51d81b7aced65f6958c78ad3dceba0a3f6d9f (diff) | |
| download | bcm5719-llvm-caa9619bcef4e67f5c2d4603b1d67c4cca6cacd8.tar.gz bcm5719-llvm-caa9619bcef4e67f5c2d4603b1d67c4cca6cacd8.zip | |
[sanitizer] Make __sanitizer::CheckFailed not public
Summary:
As far as I can tell, there is no reason why `__sanitizer::CheckFailed` should
be exported. Looking back in time, it was added with the FIXME with the
following by @timurrrr:
```
[*San/RTL] Fix minor breakage
Grumbling: this hasn't been caught by running 'make check-{a,l,t}san check-sanitizer'
```
I can't find any detail about the breakage, all tests seem to work for me, so
maybe Windows (@rnk?) or something I have no setup for.
The reason to make it private (past the FIXME) is that Scudo defines its own
(without callback) and I am trying to make the .so be loadable with the UBsan
one (that has its own public `CheckFailed`) with as little drama as possible.
Reviewers: eugenis, rnk
Reviewed By: eugenis, rnk
Subscribers: kubamracek, delcypher, #sanitizers, timurrrr, rnk, llvm-commits
Differential Revision: https://reviews.llvm.org/D52279
llvm-svn: 342651
Diffstat (limited to 'llvm')
0 files changed, 0 insertions, 0 deletions
