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

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 17:45:28 PST 2024


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

Just did a quick glance on this, another problem in that expression is that we may want to change it from
```
sizeof(StackDepot) + UINT32_MAX * sizeof(atomic_u64) * UINT32_MAX * sizeof(atomic_u32)
```
to
```
sizeof(StackDepot) + UINT32_MAX * sizeof(atomic_u64)  + UINT32_MAX * sizeof(atomic_u32)
```
which change the middle "*" to "+". But still, the UINT32_MAX may not be a good boundary. 

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


More information about the llvm-commits mailing list