[compiler-rt] Sync FuchsiaConfig with downstream's custom_scudo_config.h (PR #89244)

Fabio D'Urso via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 07:38:29 PDT 2024


https://github.com/fabio-d created https://github.com/llvm/llvm-project/pull/89244

Downstream disabled EnableContiguousRegions on RISCV-64 to avoid
running out of virtual memory, but our tests still use the internal
FuchsiaConfig class, which therefore needs to be changed too.


>From 16a29ce8b854f8f6af36ee6990e2eca3a5d96df0 Mon Sep 17 00:00:00 2001
From: Fabio D'Urso <fdurso at google.com>
Date: Thu, 18 Apr 2024 16:03:52 +0200
Subject: [PATCH] Sync FuchsiaConfig with downstream's custom_scudo_config.h

Downstream disabled EnableContiguousRegions on RISCV-64 to avoid
running out of virtual memory, but our tests still use the internal
FuchsiaConfig class, which therefore needs to be changed too.
---
 compiler-rt/lib/scudo/standalone/allocator_config.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/compiler-rt/lib/scudo/standalone/allocator_config.h b/compiler-rt/lib/scudo/standalone/allocator_config.h
index 1e0cf1015ba67e..0a89b4bddfa7c6 100644
--- a/compiler-rt/lib/scudo/standalone/allocator_config.h
+++ b/compiler-rt/lib/scudo/standalone/allocator_config.h
@@ -146,9 +146,11 @@ struct FuchsiaConfig {
     // Support 39-bit VMA for riscv-64
     static const uptr RegionSizeLog = 28U;
     static const uptr GroupSizeLog = 19U;
+    static const bool EnableContiguousRegions = false;
 #else
     static const uptr RegionSizeLog = 30U;
     static const uptr GroupSizeLog = 21U;
+    static const bool EnableContiguousRegions = true;
 #endif
     typedef u32 CompactPtrT;
     static const bool EnableRandomOffset = true;



More information about the llvm-commits mailing list