[PATCH] D27722: [GVNHoist] Move GVNHoist to function simplification part of pipeline.
Geoff Berry via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 14 11:48:46 PST 2016
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289696: [GVNHoist] Move GVNHoist to function simplification part of pipeline. (authored by gberry).
Changed prior to commit:
https://reviews.llvm.org/D27722?vs=81261&id=81432#toc
Repository:
rL LLVM
https://reviews.llvm.org/D27722
Files:
llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
llvm/trunk/test/Transforms/GVNHoist/hoist-inline.ll
Index: llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
===================================================================
--- llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
+++ llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
@@ -249,8 +249,6 @@
FPM.add(createCFGSimplificationPass());
FPM.add(createSROAPass());
FPM.add(createEarlyCSEPass());
- if(EnableGVNHoist)
- FPM.add(createGVNHoistPass());
FPM.add(createLowerExpectIntrinsicPass());
}
@@ -295,6 +293,8 @@
// Break up aggregate allocas, using SSAUpdater.
MPM.add(createSROAPass());
MPM.add(createEarlyCSEPass()); // Catch trivial redundancies
+ if(EnableGVNHoist)
+ MPM.add(createGVNHoistPass());
// Speculative execution if the target has divergent branches; otherwise nop.
MPM.add(createSpeculativeExecutionIfHasBranchDivergencePass());
MPM.add(createJumpThreadingPass()); // Thread jumps.
Index: llvm/trunk/test/Transforms/GVNHoist/hoist-inline.ll
===================================================================
--- llvm/trunk/test/Transforms/GVNHoist/hoist-inline.ll
+++ llvm/trunk/test/Transforms/GVNHoist/hoist-inline.ll
@@ -0,0 +1,38 @@
+; RUN: opt -S -O2 < %s | FileCheck %s
+
+; Check that the inlined loads are hoisted.
+; CHECK-LABEL: define i32 @fun(
+; CHECK-LABEL: entry:
+; CHECK: load i32, i32* @A
+; CHECK: if.then:
+
+ at A = external global i32
+ at B = external global i32
+ at C = external global i32
+
+define i32 @loadA() {
+ %a = load i32, i32* @A
+ ret i32 %a
+}
+
+define i32 @fun(i1 %c) {
+entry:
+ br i1 %c, label %if.then, label %if.else
+
+if.then:
+ store i32 1, i32* @B
+ %call1 = call i32 @loadA()
+ store i32 2, i32* @C
+ br label %if.endif
+
+if.else:
+ store i32 2, i32* @C
+ %call2 = call i32 @loadA()
+ store i32 1, i32* @B
+ br label %if.endif
+
+if.endif:
+ %ret = phi i32 [ %call1, %if.then ], [ %call2, %if.else ]
+ ret i32 %ret
+}
+
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27722.81432.patch
Type: text/x-patch
Size: 1921 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161214/5954f14f/attachment.bin>
More information about the llvm-commits
mailing list