diff options
author | Mingming Liu <mingmingl@google.com> | 2024-02-07 20:03:44 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-07 20:03:44 -0800 |
commit | 05091aa3ac53a13d08c78882c0c2035e58a1b4c4 (patch) | |
tree | e1874996ddf2aa94d853812ab87a69cd3e12b934 /llvm/unittests/ProfileData/InstrProfTest.cpp | |
parent | d01864eb2f21d56cf432da7d80c505f510533c46 (diff) | |
download | llvm-05091aa3ac53a13d08c78882c0c2035e58a1b4c4.zip llvm-05091aa3ac53a13d08c78882c0c2035e58a1b4c4.tar.gz llvm-05091aa3ac53a13d08c78882c0c2035e58a1b4c4.tar.bz2 |
[NFC][InstrProf]Generalize getParsedIRPGOFuncName to getParsedIRPGOName (#81054)
- Function getParsedIRPGOFuncName splits name by delimiter. The `[filename;]mangled-name` format could be generalized for non-function global values (e.g., vtables for type profiling). So rename the
function.
- Use kGlobalIdentifierDelimiter rather than semicolon directly for defragmentation.
Diffstat (limited to 'llvm/unittests/ProfileData/InstrProfTest.cpp')
-rw-r--r-- | llvm/unittests/ProfileData/InstrProfTest.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/llvm/unittests/ProfileData/InstrProfTest.cpp b/llvm/unittests/ProfileData/InstrProfTest.cpp index 8ffb68d..cd4552a 100644 --- a/llvm/unittests/ProfileData/InstrProfTest.cpp +++ b/llvm/unittests/ProfileData/InstrProfTest.cpp @@ -561,8 +561,7 @@ TEST_F(InstrProfTest, test_irpgo_function_name) { auto IRPGOFuncName = getIRPGOFuncName(*F); EXPECT_EQ(IRPGOFuncName, ExpectedIRPGOFuncName); - auto [Filename, ParsedIRPGOFuncName] = - getParsedIRPGOFuncName(IRPGOFuncName); + auto [Filename, ParsedIRPGOFuncName] = getParsedIRPGOName(IRPGOFuncName); StringRef ExpectedParsedIRPGOFuncName = IRPGOFuncName; if (ExpectedParsedIRPGOFuncName.consume_front("MyModule.cpp;")) { EXPECT_EQ(Filename, "MyModule.cpp"); @@ -1279,8 +1278,7 @@ TEST(SymtabTest, instr_prof_symtab_module_test) { auto IRPGOFuncName = ProfSymtab.getFuncOrVarName(IndexedInstrProf::ComputeHash(IRPGOName)); EXPECT_EQ(StringRef(IRPGOName), IRPGOFuncName); - EXPECT_EQ(StringRef(Funcs[I]), - getParsedIRPGOFuncName(IRPGOFuncName).second); + EXPECT_EQ(StringRef(Funcs[I]), getParsedIRPGOName(IRPGOFuncName).second); // Ensure we can still read this old record name. std::string PGOName = getPGOFuncName(*F); auto PGOFuncName = |