[compiler-rt] [scudo] Allow to resize allocation ring buffer (PR #82683)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 15:19:01 PST 2024


================
@@ -1564,11 +1570,11 @@ class Allocator {
                   UINTPTR_MAX);
----------------
ChiaHungDuan wrote:

There's nothing that relates to any constexpr arguments or variables here. That's the constraint to the `StackDepot` (especially the ` static_assert(sizeof(StackDepot) % alignof(atomic_u64) == 0)`)

Besides, I'm not sure if the overflow is intended,
```
UINT32_MAX * sizeof(atomic_u64) * UINT32_MAX * sizeof(atomic_u32)
```
I guess the assertion will never be triggered. 

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


More information about the llvm-commits mailing list