On Tue Feb 18 23:58:22 2025 +0000, Paul Gofman wrote:
What does a DAG have to do with this? The dependency graph isn’t
acyclic. It can’t be, since circular dependencies are allowed. We already create cycles for regular imports, so I don’t see why forward imports should be treated any differently. Not sure if that is directly related, but I think the idea is that modules with cyclic dependencies should exist in one DDAG node while there should not be cyclic dependencies between DDAG nodes. E. g., user32, win32u and gdi32 get loaded in one DDAG node on Windows. This is currently not implemented and we always create a separate DDAG node for each module.
Rebased, `LDR_DONT_RESOLVE_REFS` test is no longer part of the patch after !7347.