Skip to content

Commit

Permalink
[memprof] Remove getFullSchema in MemProfTest.cpp (#90072)
Browse files Browse the repository at this point in the history
This patch removes getFullSchema in MemProfTest.cpp in favor of
llvm::memprof::PortableMemInfoBlock::getFullSchema as they do exactly
the same thing.
  • Loading branch information
kazutakahirata committed Apr 25, 2024
1 parent 39adc8f commit f9a0b46
Showing 1 changed file with 3 additions and 11 deletions.
14 changes: 3 additions & 11 deletions llvm/unittests/ProfileData/MemProfTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -122,14 +122,6 @@ MATCHER_P4(FrameContains, FunctionName, LineOffset, Column, Inline, "") {
return false;
}

MemProfSchema getFullSchema() {
MemProfSchema Schema;
#define MIBEntryDef(NameTag, Name, Type) Schema.push_back(Meta::Name);
#include "llvm/ProfileData/MIBEntryDef.inc"
#undef MIBEntryDef
return Schema;
}

TEST(MemProf, FillsValue) {
std::unique_ptr<MockSymbolizer> Symbolizer(new MockSymbolizer());

Expand Down Expand Up @@ -248,7 +240,7 @@ TEST(MemProf, PortableWrapper) {
/*dealloc_timestamp=*/2000, /*alloc_cpu=*/3,
/*dealloc_cpu=*/4);

const auto Schema = getFullSchema();
const auto Schema = llvm::memprof::PortableMemInfoBlock::getFullSchema();
PortableMemInfoBlock WriteBlock(Info);

std::string Buffer;
Expand All @@ -271,7 +263,7 @@ TEST(MemProf, PortableWrapper) {
// Version0 and Version1 serialize IndexedMemProfRecord in the same format, so
// we share one test.
TEST(MemProf, RecordSerializationRoundTripVersion0And1) {
const MemProfSchema Schema = getFullSchema();
const auto Schema = llvm::memprof::PortableMemInfoBlock::getFullSchema();

MemInfoBlock Info(/*size=*/16, /*access_count=*/7, /*alloc_timestamp=*/1000,
/*dealloc_timestamp=*/2000, /*alloc_cpu=*/3,
Expand Down Expand Up @@ -305,7 +297,7 @@ TEST(MemProf, RecordSerializationRoundTripVersion0And1) {
}

TEST(MemProf, RecordSerializationRoundTripVerion2) {
const MemProfSchema Schema = getFullSchema();
const auto Schema = llvm::memprof::PortableMemInfoBlock::getFullSchema();

MemInfoBlock Info(/*size=*/16, /*access_count=*/7, /*alloc_timestamp=*/1000,
/*dealloc_timestamp=*/2000, /*alloc_cpu=*/3,
Expand Down

0 comments on commit f9a0b46

Please sign in to comment.