[llvm-branch-commits] [llvm] [BOLT] Match blocks with calls as anchors (PR #96596)

Shaw Young via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Mon Jul 8 11:08:17 PDT 2024


================
@@ -35,6 +36,12 @@ std::string hashBlock(BinaryContext &BC, const BinaryBasicBlock &BB,
 
 std::string hashBlockLoose(BinaryContext &BC, const BinaryBasicBlock &BB);
 
+std::string hashBlockCalls(BinaryContext &BC, const BinaryBasicBlock &BB);
+
+std::string
+hashBlockCalls(const DenseMap<uint32_t, std::string *> &IdToFunctionName,
----------------
shawbyoung wrote:

On second thought after some experimentation, references in maps (both for std and llvm) are a bit tricky to deal with and result in needing particular idioms to not break the map. (https://stackoverflow.com/questions/1543193/why-cant-i-store-references-in-a-stdmap-in-c) Pointers seem to be the better choice 

https://github.com/llvm/llvm-project/pull/96596


More information about the llvm-branch-commits mailing list