[compiler-rt] [NFC] [scudo] turn DCHECK into CHECK (PR #84255)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 6 15:06:30 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: Florian Mayer (fmayer)
<details>
<summary>Changes</summary>
This should get optimized away anyway: https://godbolt.org/z/GM6sKM5YE
---
Full diff: https://github.com/llvm/llvm-project/pull/84255.diff
1 Files Affected:
- (modified) compiler-rt/lib/scudo/standalone/combined.h (+1-1)
``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/combined.h b/compiler-rt/lib/scudo/standalone/combined.h
index fa6077384d9826..10c2f5ffb90a90 100644
--- a/compiler-rt/lib/scudo/standalone/combined.h
+++ b/compiler-rt/lib/scudo/standalone/combined.h
@@ -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));
uptr StackDepotSize = sizeof(StackDepot) + sizeof(atomic_u64) * RingSize +
sizeof(atomic_u32) * TabSize;
``````````
</details>
https://github.com/llvm/llvm-project/pull/84255
More information about the llvm-commits
mailing list