[lldb] Remove orphaned modules in a loop
authorRaphael Isemann <teemperor@gmail.com>
Mon, 20 Jul 2020 08:47:21 +0000 (10:47 +0200)
committerRaphael Isemann <teemperor@gmail.com>
Mon, 20 Jul 2020 08:47:56 +0000 (10:47 +0200)
commit139e2a3f7b279ca886c71b676f983ea314c36568
tree72afaf24cdfb7c227dfaa9fa23377d9881f644e9
parent256aea816da45bd3bc52317cbc89e704095a8991
[lldb] Remove orphaned modules in a loop

Summary:

When modules reference each other (which happens for example with the different
modules LLDB loads when debugging -gmodules-compiled binaries), just iterating
over the module list once isn't good enough to find all orphans. Any removed
modules in the module list will also clear up the shared pointers they hold to
other modules, so after any module was removed from the list, LLDB should
iterate again and check if any additional modules can no be safely deleted.

This is currently causing that many gmodules tests are not cleaning up all
allocated modules which causes cleanup asserts to fail (right now these asserts
just mark the test as unsupported, but after D83865 the tests will start
failing).

Reviewers: aprantl, clayborg, JDevlieghere

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D84015
lldb/source/Core/ModuleList.cpp