Bug 1083686 part 0 - Don't set MOZ_MEMORY_API to static for OSX in replace_malloc.c. r=njn

It appears to be an unnecessary optimization, since the compiler is still inlining
the functions when they're not marked static. OTOH, following patches will require
the _impl functions not to be static.
This commit is contained in:
Mike Hommey 2014-10-24 13:08:01 +09:00
parent 47ec70908d
commit 746f48e4a4

View File

@ -103,17 +103,6 @@ replace_malloc_init_funcs()
* replacement functions if they exist.
*/
/*
* On OSX, MOZ_MEMORY_API is defined to nothing, because malloc functions
* are meant to have hidden visibility. But since the functions are only
* used locally in the zone allocator further below, we can allow the
* compiler to optimize more by switching to static.
*/
#ifdef XP_DARWIN
#undef MOZ_MEMORY_API
#define MOZ_MEMORY_API static
#endif
/*
* Malloc implementation functions are MOZ_MEMORY_API, and jemalloc
* specific functions MOZ_JEMALLOC_API; see mozmemory_wrap.h