[PATCH] D40146: [JumpThreading] Preservation of DT and LVI across the pass

[Brian Rzycki via Phabricator via llvm-commits]

brzycki added a subscriber: rsmith.
brzycki added a comment.

> I wonder what caused the original cycle. I'm not entirely sure how modules are built, but have you tried getting rid of the include in Local.h? If that's not enough to break the cycle, I don't have any idea better than yours.

It's a bit of a mystery to me how -fmodules actually works. I am getting cyclic errors when compiling a file that does not include either of Dominators.h or DeferredDominance.h. I have emailed @rsmith requesting his expertise if he can help us understand this.

I'm currently working on a version of the patch with no functions inlined in the header. I should have the updated diff available for review shortly.


https://reviews.llvm.org/D40146