[Lldb-commits] [PATCH] D96637: Make sure the interpreter module was loaded before making checks against it

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

I'd like @mgorny to confirm this, but I think this should be fine -- the `m_rendezvous.IsValid()` branch does not do any funky module loading, so funky **un**loading should also be unnecessary.

>From the description is seems like trigerring this bug should be relatively simple. All it takes is to attach to a process (after it has finished setting up the dynamic section). And indeed, from my simple experiments, it seems that we are indeed failing to load modules in that scenarios.

=> We should have a test for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96637