[llvm-branch-commits] [llvm] [llvm-profgen] Extend llvm-profgen to generate vtable profiles with data access events. (PR #148013)

[Wei Wang via llvm-branch-commits]

================
@@ -603,6 +640,16 @@ class ProfiledBinary {
     return ProbeDecoder.getInlinerDescForProbe(Probe);
   }
 
+  void addMMapNonTextEvent(MMapEvent MMap) {
+    MMapNonTextEvents.push_back(MMap);
----------------
apolloww wrote:

Thanks for the update! 

Sorry for not making it more clearer. I think this won't be an issue if we only consider the main executable right now. Its sections should just be loaded once and stay the same. I was thinking about if we are going to support dynamic loading/reloading of shared libraries (`dlopen/dlclose`), which may make the same DSO appear at different addresses. Maybe this isn't something we need to consider right now, the current map solution can help surface such issue in case we want to go that direction.

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