diff options
| author | Kostya Kortchinsky <kostyak@google.com> | 2018-01-17 21:54:48 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2018-01-17 21:54:48 +0000 |
| commit | 255913b3a0e03ea5f0faeff951caa5fc5c804a91 (patch) | |
| tree | 03821838324b2bf855a31bef181548eba5a92902 /llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp | |
| parent | fe8e00bc3c65115b2e3d2a43cf3d0d756a934a52 (diff) | |
| download | bcm5719-llvm-255913b3a0e03ea5f0faeff951caa5fc5c804a91.tar.gz bcm5719-llvm-255913b3a0e03ea5f0faeff951caa5fc5c804a91.zip | |
[scudo] Limit by default the TSD pool to 2 on Android
Summary:
jemalloc on Android currently uses 2 arenas
(https://android.googlesource.com/platform/external/jemalloc/+/master/Android.bp#64).
Since the Android toolchain absorbs compiler-rt and compiles it as is, we have
to enforce the same limit to somehow stay competitive in terms of memory usage.
The changes could either go in:
- `scudo_platform.h` with a default for Android of 2 (this is the solution
implemented here);
- in `CMakeLists.txt` adding -DSCUDO_SHARED_TSD_POOL_SIZE=2 for Android.
- something else?
I don't have a strong opinion on how to do it, but it has to be done upstream
anyway.
Reviewers: alekseyshl, eugenis
Reviewed By: alekseyshl, eugenis
Subscribers: srhines, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D42194
llvm-svn: 322764
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp')
0 files changed, 0 insertions, 0 deletions
