Skip to content

Commit 40a5a86

Browse files
[ProfileData] Migrate to a new version of getValueProfDataFromInst (#96381)
1 parent ed95844 commit 40a5a86

File tree

1 file changed

+10
-16
lines changed

1 file changed

+10
-16
lines changed

llvm/unittests/ProfileData/InstrProfTest.cpp

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -942,12 +942,10 @@ TEST_P(MaybeSparseInstrProfTest, annotate_vp_data) {
942942
Instruction *Inst2 = Builder.CreateCondBr(Builder.getTrue(), TBB, FBB);
943943
annotateValueSite(*M, *Inst, R.get(), IPVK_IndirectCallTarget, 0);
944944

945-
uint32_t N;
946945
uint64_t T;
947946
auto ValueData =
948-
getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, N, T);
949-
ASSERT_NE(ValueData, nullptr);
950-
ASSERT_EQ(3U, N);
947+
getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, T);
948+
ASSERT_THAT(ValueData, SizeIs(3));
951949
ASSERT_EQ(21U, T);
952950
// The result should be sorted already:
953951
ASSERT_EQ(6000U, ValueData[0].Value);
@@ -956,22 +954,19 @@ TEST_P(MaybeSparseInstrProfTest, annotate_vp_data) {
956954
ASSERT_EQ(5U, ValueData[1].Count);
957955
ASSERT_EQ(4000U, ValueData[2].Value);
958956
ASSERT_EQ(4U, ValueData[2].Count);
959-
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 1, N, T);
960-
ASSERT_NE(ValueData, nullptr);
961-
ASSERT_EQ(1U, N);
957+
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 1, T);
958+
ASSERT_THAT(ValueData, SizeIs(1));
962959
ASSERT_EQ(21U, T);
963960

964-
ValueData =
965-
getValueProfDataFromInst(*Inst2, IPVK_IndirectCallTarget, 5, N, T);
966-
ASSERT_EQ(ValueData, nullptr);
961+
ValueData = getValueProfDataFromInst(*Inst2, IPVK_IndirectCallTarget, 5, T);
962+
ASSERT_THAT(ValueData, SizeIs(0));
967963

968964
// Remove the MD_prof metadata
969965
Inst->setMetadata(LLVMContext::MD_prof, 0);
970966
// Annotate 5 records this time.
971967
annotateValueSite(*M, *Inst, R.get(), IPVK_IndirectCallTarget, 0, 5);
972-
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, N, T);
973-
ASSERT_NE(ValueData, nullptr);
974-
ASSERT_EQ(5U, N);
968+
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, T);
969+
ASSERT_THAT(ValueData, SizeIs(5));
975970
ASSERT_EQ(21U, T);
976971
ASSERT_EQ(6000U, ValueData[0].Value);
977972
ASSERT_EQ(6U, ValueData[0].Count);
@@ -991,9 +986,8 @@ TEST_P(MaybeSparseInstrProfTest, annotate_vp_data) {
991986
{5000, 2}, {6000, 1}};
992987
annotateValueSite(*M, *Inst, ArrayRef(VD0Sorted).slice(2), 10,
993988
IPVK_IndirectCallTarget, 5);
994-
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, N, T);
995-
ASSERT_NE(ValueData, nullptr);
996-
ASSERT_EQ(4U, N);
989+
ValueData = getValueProfDataFromInst(*Inst, IPVK_IndirectCallTarget, 5, T);
990+
ASSERT_THAT(ValueData, SizeIs(4));
997991
ASSERT_EQ(10U, T);
998992
ASSERT_EQ(3000U, ValueData[0].Value);
999993
ASSERT_EQ(4U, ValueData[0].Count);

0 commit comments

Comments
 (0)