[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