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

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to