[compiler-rt] [NFC] [scudo] turn DCHECK into CHECK (PR #84255)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 15:27:08 PST 2024


================
@@ -1570,7 +1570,7 @@ class Allocator {
     if (TabSize > UINT32_MAX / kFramesPerStack)
       return;
     u32 RingSize = static_cast<u32>(TabSize * kFramesPerStack);
-    DCHECK(isPowerOfTwo(RingSize));
+    CHECK(isPowerOfTwo(RingSize));
 
----------------
ChiaHungDuan wrote:

I think we can remove the DCHECK. It has been verified in the StackDepot::init() and I think it makes more sense to check it there because it uses the value.

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


More information about the llvm-commits mailing list