[compiler-rt] [scudo] Allow to resize allocation ring buffer (PR #82683)
    Florian Mayer via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Mar  6 15:06:16 PST 2024
    
    
  
================
@@ -1564,11 +1570,11 @@ class Allocator {
                   UINTPTR_MAX);
 
     if (AllocationRingBufferSize > kMaxU32Pow2 / kStacksPerRingBufferEntry)
-      return;
+      return false;
     u32 TabSize = static_cast<u32>(roundUpPowerOfTwo(kStacksPerRingBufferEntry *
                                                      AllocationRingBufferSize));
     if (TabSize > UINT32_MAX / kFramesPerStack)
-      return;
+      return false;
     u32 RingSize = static_cast<u32>(TabSize * kFramesPerStack);
     DCHECK(isPowerOfTwo(RingSize));
----------------
fmayer wrote:
Sure (seems like clang knows to optimize this away anyway https://godbolt.org/z/GM6sKM5YE): https://github.com/llvm/llvm-project/pull/84255
https://github.com/llvm/llvm-project/pull/82683
    
    
More information about the llvm-commits
mailing list