[PATCH] D37528: [JumpThreading] Preserve dominance across the pass.

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 7 16:53:21 PDT 2017


brzycki marked 10 inline comments as done.
brzycki added a comment.

In https://reviews.llvm.org/D37528#863731, @kuhar wrote:

> @brzycki, one related thing: have you tried preserving postdominators as well? I'm not sure if that would be beneficial here, but in theory, it should be pretty simple, as you can pass exactly the same update sequences to the DominatorTree and PostDominatorTree.


I have not. I'll talk to @sebpop and see if he thinks this makes sense to include in this patch. Depending on our follow-on updates to JT we may need post dominators as well.


https://reviews.llvm.org/D37528





More information about the llvm-commits mailing list