================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to