For reproducers in LLDB we want to hook up into the existing clang infrastructure. To make that happen we need to be able to override the ModuleDependencyCollector's methods.
We reuse the attachTo methods, which in turn use the ModuleDependencyListener, ModuleDependencyPPCallbacks and ModuleDependencyMMCallbacks. Do you think it's worth re-implementing those?
I missed that the DependencyCollector already marks them virtual, you are just making it obvious here. I think you can omit the ones that are already virtual and only add to the ones that are on the intend of this patch.