[Lldb-commits] [PATCH] D92187: [lldb] [POSIX-DYLD] Add libraries from initial rendezvous brkpt hit

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In D92187#2450526 <https://reviews.llvm.org/D92187#2450526>, @mgorny wrote:

> Another fix: we should only unload duplicate ld.so if it's actually a duplicate, i.e. the path differs. Otherwise, we've ended up unloading the only copy.

Would comparing the module shared_pointers work? I think that's exactly what we want here. I think it's possible for `LoadModuleAtAddress` to return the same module, even with paths which don't match exactly. But if it does that, then we will still end up unloading the original interpreter module...


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92187/new/

https://reviews.llvm.org/D92187