diff options
| author | Kostya Kortchinsky <kostyak@google.com> | 2018-01-17 23:10:02 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2018-01-17 23:10:02 +0000 |
| commit | 33802be5791fd085518eaad60f964b868fbdf473 (patch) | |
| tree | e017a01fff9995a55d7b575c7e1bca46049b8c7f /llvm/lib/CodeGen/MachinePostDominators.cpp | |
| parent | c8032c623300dac2c47cf3c747529046b3ba1f44 (diff) | |
| download | bcm5719-llvm-33802be5791fd085518eaad60f964b868fbdf473.tar.gz bcm5719-llvm-33802be5791fd085518eaad60f964b868fbdf473.zip | |
[scudo] Fix for the Scudo interface function scope
Summary:
A forgotten include in `scudo_allocator.cpp` made the symbol only local :/
Before:
```
nm ./lib/clang/7.0.0/lib/linux/libclang_rt.scudo-i686-android.so | grep rss
00024730 t __scudo_set_rss_limit
```
After:
```
nm ./lib/clang/7.0.0/lib/linux/libclang_rt.scudo-i686-android.so | grep rs
00024760 T __scudo_set_rss_limit
```
And we want `T`!
This include also means that we can get rid of the `extern "C"` in the C++
file, the compiler does fine without it (note that this was already the case
for all the `__sanitizer_*` interface functions.
Reviewers: alekseyshl, eugenis
Reviewed By: eugenis
Subscribers: #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D42199
llvm-svn: 322782
Diffstat (limited to 'llvm/lib/CodeGen/MachinePostDominators.cpp')
0 files changed, 0 insertions, 0 deletions

