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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 6 14:33:49 PDT 2017


davide added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/JumpThreading.cpp:107-108
       AU.addRequired<TargetLibraryInfoWrapperPass>();
+      AU.addRequired<DominatorTreeWrapperPass>();
+      AU.addPreserved<DominatorTreeWrapperPass>();
     }
----------------
Now that the domtree is preserved, you can probably also preserve LVI.
The reason why it was disabled is that LVI holds a pointer to DomTree, and when you free the dominator analysis result, you end up with garbage.


Repository:
  rL LLVM

https://reviews.llvm.org/D37528





More information about the llvm-commits mailing list