[compiler-rt] Sync FuchsiaConfig with downstream's custom_scudo_config.h (PR #89244)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 18 07:39:00 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: Fabio D'Urso (fabio-d)
<details>
<summary>Changes</summary>
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.
---
Full diff: https://github.com/llvm/llvm-project/pull/89244.diff
1 Files Affected:
- (modified) compiler-rt/lib/scudo/standalone/allocator_config.h (+2)
``````````diff
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;
``````````
</details>
https://github.com/llvm/llvm-project/pull/89244
More information about the llvm-commits
mailing list