[PATCH] D43007: Add missed PostDominatorTree analysis dependency to GVN hoist pass.

Arkady Shlykov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 02:15:20 PST 2018


ashlykov created this revision.
ashlykov added reviewers: hiraditya, sebpop.
Herald added a reviewer: dberlin.

GVN hoist pass is using PostDominatorTree analysis, therefore the analysis
should be listed in the pass initialization as a dependency.


Repository:
  rL LLVM

https://reviews.llvm.org/D43007

Files:
  lib/Transforms/Scalar/GVNHoist.cpp


Index: lib/Transforms/Scalar/GVNHoist.cpp
===================================================================
--- lib/Transforms/Scalar/GVNHoist.cpp
+++ lib/Transforms/Scalar/GVNHoist.cpp
@@ -1200,6 +1200,7 @@
 INITIALIZE_PASS_DEPENDENCY(MemoryDependenceWrapperPass)
 INITIALIZE_PASS_DEPENDENCY(MemorySSAWrapperPass)
 INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
+INITIALIZE_PASS_DEPENDENCY(PostDominatorTreeWrapperPass)
 INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass)
 INITIALIZE_PASS_END(GVNHoistLegacyPass, "gvn-hoist",
                     "Early GVN Hoisting of Expressions", false, false)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43007.133177.patch
Type: text/x-patch
Size: 605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180207/36a7d1e2/attachment.bin>


More information about the llvm-commits mailing list