[compiler-rt] [llvm] [InstrProf] Add vtables with type metadata into symtab (PR #81051)
via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 13:57:02 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 45eabd1362527d1b3a27a90f7479865785b763ee 6422502ec2fa6143506517c60b86b842c271ba62 -- llvm/include/llvm/ProfileData/InstrProf.h llvm/lib/ProfileData/InstrProf.cpp llvm/unittests/ProfileData/InstrProfTest.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/include/llvm/ProfileData/InstrProf.h b/llvm/include/llvm/ProfileData/InstrProf.h
index f12ba61dbe..88c7fe425b 100644
--- a/llvm/include/llvm/ProfileData/InstrProf.h
+++ b/llvm/include/llvm/ProfileData/InstrProf.h
@@ -476,7 +476,7 @@ private:
// `MD5FuncMap` and `MD5VTableMap`.
// TODO: Unify the container type and the lambda function 'mapName' inside
// add{Func,VTable}WithName.
- DenseMap<uint64_t, GlobalVariable*> MD5VTableMap;
+ DenseMap<uint64_t, GlobalVariable *> MD5VTableMap;
// A map from function runtime address to function name MD5 hash.
// This map is only populated and used by raw instr profile reader.
AddrHashMap AddrToMD5Map;
@@ -646,7 +646,7 @@ public:
/// Return the global variable corresponding to md5 hash. Return nullptr if
/// not found.
- inline GlobalVariable* getGlobalVariable(uint64_t MD5Hash);
+ inline GlobalVariable *getGlobalVariable(uint64_t MD5Hash);
/// Return the name section data.
inline StringRef getNameData() const { return Data; }
@@ -727,7 +727,7 @@ Function* InstrProfSymtab::getFunction(uint64_t FuncMD5Hash) {
return nullptr;
}
-GlobalVariable* InstrProfSymtab::getGlobalVariable(uint64_t MD5Hash) {
+GlobalVariable *InstrProfSymtab::getGlobalVariable(uint64_t MD5Hash) {
if (auto Iter = MD5VTableMap.find(MD5Hash); Iter != MD5VTableMap.end())
return Iter->second;
return nullptr;
diff --git a/llvm/lib/ProfileData/InstrProf.cpp b/llvm/lib/ProfileData/InstrProf.cpp
index 655e6cebcf..39d275c7b5 100644
--- a/llvm/lib/ProfileData/InstrProf.cpp
+++ b/llvm/lib/ProfileData/InstrProf.cpp
@@ -481,8 +481,8 @@ Error InstrProfSymtab::create(Module &M, bool InLTO) {
if (!G.hasName() || !G.hasMetadata(LLVMContext::MD_type))
continue;
if (Error E = addVTableWithName(
- G, getIRPGOObjectName(G, InLTO, /* PGONameMetadata */ nullptr)))
- return E;
+ G, getIRPGOObjectName(G, InLTO, /* PGONameMetadata */ nullptr)))
+ return E;
}
Sorted = false;
@@ -497,7 +497,8 @@ Error InstrProfSymtab::addVTableWithName(GlobalVariable &VTable,
return E;
bool Inserted = true;
- std::tie(std::ignore, Inserted) = MD5VTableMap.try_emplace(GlobalValue::getGUID(Name), &VTable);
+ std::tie(std::ignore, Inserted) =
+ MD5VTableMap.try_emplace(GlobalValue::getGUID(Name), &VTable);
if (!Inserted)
LLVM_DEBUG(dbgs() << "GUID conflict within one module");
return Error::success();
``````````
</details>
https://github.com/llvm/llvm-project/pull/81051
More information about the llvm-commits
mailing list