[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