Skip to content

Commit

Permalink
[memprof] Fix builds under EXPENSIVE_CHECKS
Browse files Browse the repository at this point in the history
memprof::Version1 has been removed, so the whole block of code is
dead.
  • Loading branch information
kazutakahirata committed Nov 23, 2024
1 parent 5802367 commit a0153ea
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions llvm/lib/ProfileData/InstrProfReader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1345,18 +1345,6 @@ Error IndexedMemProfReader::deserialize(const unsigned char *Start,
break;
}

#ifdef EXPENSIVE_CHECKS
// Go through all the records and verify that CSId has been correctly
// populated. Do this only under EXPENSIVE_CHECKS. Otherwise, we
// would defeat the purpose of OnDiskIterableChainedHashTable.
// Note that we can compare CSId against actual call stacks only for
// Version0 and Version1 because IndexedAllocationInfo::CallStack and
// IndexedMemProfRecord::CallSites are not populated in Version2.
if (Version <= memprof::Version1)
for (const auto &Record : MemProfRecordTable->data())
verifyIndexedMemProfRecord(Record);
#endif

return Error::success();
}

Expand Down

0 comments on commit a0153ea

Please sign in to comment.