[compiler-rt] [scudo] limit VMA size for riscv64 for DefaultConfig (PR #76013)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 13:01:12 PST 2024


================
@@ -124,8 +124,14 @@ struct DefaultConfig {
   struct Primary {
     using SizeClassMap = DefaultSizeClassMap;
 #if SCUDO_CAN_USE_PRIMARY64
+#if SCUDO_RISCV64
+    // Support 39-bit VMA for riscv-64
+    static const uptr RegionSizeLog = 30U;
+    static const uptr GroupSizeLog = 19U;
+#else
----------------
ChiaHungDuan wrote:

Fuchsia has the default `RegionSizeLog=30` and `RegionSizeLog=28` for RISCV. There're other differences in the config but I want to know if `30` is smaller enough for RISCV or we should consider `28` as well. BTW, I think this is only for running tests, right? 

https://github.com/llvm/llvm-project/pull/76013


More information about the llvm-commits mailing list