[Lldb-commits] [lldb] [lldb] Fix deadlock in parallel module loading with separate symbol thread pool (PR #160225)

[Jonas Devlieghere via lldb-commits]

https://github.com/JDevlieghere requested changes to this pull request.

Can you explain how having a separate thread pool addresses the issue? It's unclear to me how that would avoid the deadlock. 

Regardless, I don't think we should have two thread pools as that pretty much defeats the purpose of using a pool in the first place. I think we should address the architectural issue. Would changing the ModuleList to use a R/W lock address this problem? That's something that has repeatedly come up in the past.

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