luau-lang/luau
1
0
Fork 0
mirror of https://github.com/luau-lang/luau.git synced 2024-12-13 21:40:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Patch the test for now to work with 16K pages

Browse source 
This exposes a bug in CodeAllocator that needs to be fixed properly
This commit is contained in:
Arseny Kapoulkine 2022-09-08 14:58:20 -07:00
parent 71df988dec
commit 6c708975d3
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
tests/CodeAllocator.test.cpp
View file

@ -41,8 +41,8 @@ TEST_CASE("CodeAllocation")
TEST_CASE("CodeAllocationFailure") TEST_CASE("CodeAllocationFailure")
{ {
size_t blockSize = 4096; size_t blockSize = 16384;
size_t maxTotalSize = 8192; size_t maxTotalSize = 32768;
CodeAllocator allocator(blockSize, maxTotalSize); CodeAllocator allocator(blockSize, maxTotalSize);
uint8_t* nativeData; uint8_t* nativeData;
@ -50,11 +50,13 @@ TEST_CASE("CodeAllocationFailure")
uint8_t* nativeEntry; uint8_t* nativeEntry;
std::vector<uint8_t> code; std::vector<uint8_t> code;
code.resize(6000); code.resize(18000);
// allocation has to fit in a block
REQUIRE(!allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry)); REQUIRE(!allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry));
code.resize(3000); // each allocation exhausts a block, so third allocation fails
code.resize(10000);
REQUIRE(allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry)); REQUIRE(allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry));
REQUIRE(allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry)); REQUIRE(allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry));
REQUIRE(!allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry)); REQUIRE(!allocator.allocate(nullptr, 0, code.data(), code.size(), nativeData, sizeNativeData, nativeEntry));