Revert "[Support] Recycler: Enforce minimum allocation size" (#121735)

Reverts llvm/llvm-project#121425
This commit is contained in:
Akshat Oke
2025-01-06 13:50:31 +05:30
committed by GitHub
parent 5f7568a32c
commit b51a082e1a
3 changed files with 0 additions and 49 deletions

View File

@@ -85,8 +85,6 @@ public:
"Recycler allocation alignment is less than object align!");
static_assert(sizeof(SubClass) <= Size,
"Recycler allocation size is less than object size!");
static_assert(Size >= sizeof(FreeNode) &&
"Recycler size must be at least sizeof(FreeNode)");
return FreeList ? reinterpret_cast<SubClass *>(pop_val())
: static_cast<SubClass *>(Allocator.Allocate(Size, Align));
}

View File

@@ -69,7 +69,6 @@ add_llvm_unittest(SupportTests
PerThreadBumpPtrAllocatorTest.cpp
ProcessTest.cpp
ProgramTest.cpp
RecyclerTest.cpp
RegexTest.cpp
ReverseIterationTest.cpp
ReplaceFileTest.cpp

View File

@@ -1,46 +0,0 @@
//===--- unittest/Support/RecyclerTest.cpp --------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
#include "llvm/Support/Recycler.h"
#include "llvm/Support/AllocatorBase.h"
#include "gtest/gtest.h"
using namespace llvm;
namespace {
struct Object1 {
char Data[1];
};
class DecoratedMallocAllocator : public MallocAllocator {
public:
int DeallocCount = 0;
template <typename T> void Deallocate(T *Ptr) {
DeallocCount++;
MallocAllocator::Deallocate(Ptr);
}
};
TEST(RecyclerTest, RecycleAllocation) {
DecoratedMallocAllocator Allocator;
// Recycler needs size to be atleast 8 bytes.
Recycler<Object1, 8, 8> R;
Object1 *A1 = R.Allocate(Allocator);
Object1 *A2 = R.Allocate(Allocator);
R.Deallocate(Allocator, A2);
Object1 *A3 = R.Allocate(Allocator);
EXPECT_EQ(A2, A3); // reuse the deallocated object.
R.Deallocate(Allocator, A1);
R.Deallocate(Allocator, A3);
R.clear(Allocator); // Should deallocate A1 and A3.
EXPECT_EQ(Allocator.DeallocCount, 2);
}
} // end anonymous namespace