Revert "[Support] Recycler: Enforce minimum allocation size" (#121735)
Reverts llvm/llvm-project#121425
This commit is contained in:
@@ -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));
|
||||
}
|
||||
|
||||
@@ -69,7 +69,6 @@ add_llvm_unittest(SupportTests
|
||||
PerThreadBumpPtrAllocatorTest.cpp
|
||||
ProcessTest.cpp
|
||||
ProgramTest.cpp
|
||||
RecyclerTest.cpp
|
||||
RegexTest.cpp
|
||||
ReverseIterationTest.cpp
|
||||
ReplaceFileTest.cpp
|
||||
|
||||
@@ -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
|
||||
Reference in New Issue
Block a user