Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(71)

Unified Diff: build/config/allocator.gni

Issue 2658723007: Hook up allocator shim on mac. (Closed)
Patch Set: remove a debugging test. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/trace_event/malloc_dump_provider.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/allocator.gni
diff --git a/build/config/allocator.gni b/build/config/allocator.gni
index f723e4d48ce4f6028e7e26e852ddedafc90bbb15..68f4e6b60d338386a7d2772e15c2754a5abdcd04 100644
--- a/build/config/allocator.gni
+++ b/build/config/allocator.gni
@@ -17,7 +17,7 @@ if (is_android || current_cpu == "mipsel" || is_mac || is_ios || is_asan ||
# the shim. NaCl in particular does seem to link some binaries statically
# against the debug CRT with "is_nacl=false".
if ((is_linux || is_android || (is_win && !is_component_build && !is_debug)) &&
- !is_asan && !is_lsan && !is_tsan && !is_msan) {
+ !is_asan && !is_lsan && !is_tsan && !is_msan) {
_default_use_experimental_allocator_shim = true
} else {
_default_use_experimental_allocator_shim = false
@@ -42,10 +42,12 @@ if (is_nacl) {
assert(use_allocator == "none" || use_allocator == "tcmalloc")
assert(!is_win || use_allocator == "none", "Tcmalloc doesn't work on Windows.")
+assert(!is_mac || use_allocator == "none", "Tcmalloc doesn't work on macOS.")
-assert(
- !use_experimental_allocator_shim || is_linux || is_android || is_win,
- "use_experimental_allocator_shim supported only on Linux, Android and Windows targets")
+assert(!use_experimental_allocator_shim || is_linux || is_android || is_win ||
+ is_mac,
+ "use_experimental_allocator_shim supported only on Linux, Android, " +
+ "Windows and macOS targets")
if (is_win && use_experimental_allocator_shim) {
assert(!is_component_build,
« no previous file with comments | « base/trace_event/malloc_dump_provider.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698