[clang] [clang-tools-extra] [lldb] [llvm] [Allocator] Drop RedZoneSize (non-sanitizer) and BytesAllocated members (PR #205711)

[Mikael Holmen via llvm-commits]

mikaelholmen wrote:

> Hi @MaskRay
> 
> If compiling with ASAN, this starts failing with this patch: `build-all-asan/tools/clang/unittests/AllClangUnitTests --gtest_filter=PPMemoryAllocationsTest.PPMacroDefinesAllocations`
> 
> It fails with:
> 
> ```
> Note: Google Test filter = PPMemoryAllocationsTest.PPMacroDefinesAllocations
> [==========] Running 1 test from 1 test suite.
> [----------] Global test environment set-up.
> [----------] 1 test from PPMemoryAllocationsTest
> [ RUN      ] PPMemoryAllocationsTest.PPMacroDefinesAllocations
> Preprocessor allocator memory for 1000000 #define: 136839168
> Bytes per #define: 1.368392e+02
> /repo/llvm-project/clang/unittests/Lex/PPMemoryAllocationsTest.cpp:92: Failure
> Expected: (BytesPerDefine) < (130.0f), actual: 136.839172 vs 130
> 
> [  FAILED  ] PPMemoryAllocationsTest.PPMacroDefinesAllocations (5651 ms)
> [----------] 1 test from PPMemoryAllocationsTest (5651 ms total)
> 
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test suite ran. (5652 ms total)
> [  PASSED  ] 0 tests.
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] PPMemoryAllocationsTest.PPMacroDefinesAllocations
> 
>  1 FAILED TEST
> ```
> 
> It still fails on latest trunk ([030e141](https://github.com/llvm/llvm-project/commit/030e141fa93aa13420f02fbcdc725a3ec604af19)).
> 
> Also seen in the build bot a few comments up: [#205711 (comment)](https://github.com/llvm/llvm-project/pull/205711#issuecomment-4824701543)

Ping @MaskRay : Did you see this? It still fails: https://lab.llvm.org/buildbot/#/builders/169/builds/24285

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