summaryrefslogtreecommitdiffstats
path: root/compiler-rt
diff options
context:
space:
mode:
authorKuba Brecka <kuba.brecka@gmail.com>2015-11-12 11:29:21 +0000
committerKuba Brecka <kuba.brecka@gmail.com>2015-11-12 11:29:21 +0000
commitba85da8482315fa1f5c364120004cd20dfa5c365 (patch)
tree7a2c4b45bd781805a10c155281881612f1156278 /compiler-rt
parent317e98b1a23512ffe813189d614e3c660502f755 (diff)
downloadbcm5719-llvm-ba85da8482315fa1f5c364120004cd20dfa5c365.tar.gz
bcm5719-llvm-ba85da8482315fa1f5c364120004cd20dfa5c365.zip
[asan] Remove system_malloc_zone
This seems to be dead code – `system_malloc_zone` is initialized as null and we never assign to it. This code is apparently currently never executed (in ASan), but is causes trouble for the TSan OS X port. Let's replace the checks with `COMMON_MALLOC_ENTER` (`ENSURE_ASAN_INITED()`). Differential Revision: http://reviews.llvm.org/D14334 llvm-svn: 252876
Diffstat (limited to 'compiler-rt')
-rw-r--r--compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc17
1 files changed, 3 insertions, 14 deletions
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc b/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc
index 33e9d96b8ce..17d3a1349f5 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc
@@ -29,8 +29,6 @@
// Similar code is used in Google Perftools,
// http://code.google.com/p/google-perftools.
-// TODO(glider): do we need both zones?
-static malloc_zone_t *system_malloc_zone = 0;
static malloc_zone_t sanitizer_zone;
INTERCEPTOR(malloc_zone_t *, malloc_create_zone,
@@ -155,10 +153,7 @@ size_t __sanitizer_mz_size(malloc_zone_t* zone, const void* ptr) {
extern "C"
SANITIZER_INTERFACE_ATTRIBUTE
void *__sanitizer_mz_malloc(malloc_zone_t *zone, uptr size) {
- if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) {
- CHECK(system_malloc_zone);
- return malloc_zone_malloc(system_malloc_zone, size);
- }
+ COMMON_MALLOC_ENTER();
COMMON_MALLOC_MALLOC(size);
return p;
}
@@ -184,10 +179,7 @@ void *__sanitizer_mz_calloc(malloc_zone_t *zone, size_t nmemb, size_t size) {
extern "C"
SANITIZER_INTERFACE_ATTRIBUTE
void *__sanitizer_mz_valloc(malloc_zone_t *zone, size_t size) {
- if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) {
- CHECK(system_malloc_zone);
- return malloc_zone_valloc(system_malloc_zone, size);
- }
+ COMMON_MALLOC_ENTER();
COMMON_MALLOC_VALLOC(size);
return p;
}
@@ -246,10 +238,7 @@ void __sanitizer_mz_destroy(malloc_zone_t* zone) {
extern "C"
SANITIZER_INTERFACE_ATTRIBUTE
void *__sanitizer_mz_memalign(malloc_zone_t *zone, size_t align, size_t size) {
- if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) {
- CHECK(system_malloc_zone);
- return malloc_zone_memalign(system_malloc_zone, align, size);
- }
+ COMMON_MALLOC_ENTER();
COMMON_MALLOC_MEMALIGN(align, size);
return p;
}
OpenPOWER on IntegriCloud