[compiler-rt] [scudo] Make BufferPool typed and always express sizes in terms of element count (PR #66896)
Fabio D'Urso via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 27 06:57:12 PDT 2023
================
@@ -200,22 +208,16 @@ template <uptr StaticBufferCount, uptr StaticBufferSize> class BufferPool {
class RegionPageMap {
public:
RegionPageMap()
- : Regions(0),
- NumCounters(0),
- CounterSizeBitsLog(0),
- CounterMask(0),
- PackingRatioLog(0),
- BitOffsetMask(0),
- SizePerRegion(0),
- BufferSize(0),
- Buffer(nullptr) {}
+ : Regions(0), NumCounters(0), CounterSizeBitsLog(0), CounterMask(0),
+ PackingRatioLog(0), BitOffsetMask(0), SizePerRegion(0),
+ BufferNumElements(0), Buffer(nullptr) {}
----------------
fabio-d wrote:
I did change `BufferSize` to `BufferNumElements`, and the GH presubmit checks complained:
![Screenshot from 2023-09-27 15-55-16](https://github.com/llvm/llvm-project/assets/5803506/c70db366-c5b9-40a5-a519-8aaf038d99f2)
The fix was to run clang-format, yes
https://github.com/llvm/llvm-project/pull/66896
More information about the llvm-commits
mailing list