[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