[compiler-rt] [scudo] Add primary option to enable/disable cache blocks. (PR #129794)

Jordan R AW via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 7 10:48:11 PST 2025


https://github.com/ajordanr-google commented:

Conceptually, this all makes sense to me.

The exact algorithmic details of the Batch Allocate/Deallocate are not super clear to me, but I do understand why it's useful.

The motivation of the whole PR has not quite hit me yet. I assume it's for lower memory usage during multi-threaded execution, but it's not clear from the option that this is the intent.

Otherwise my remaining comments are nits/LLVM style guide things.

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


More information about the llvm-commits mailing list