[PATCH] D17365: [LoopUnrollAnalyzer] Don't treat gep-instructions with simplified offset as simplified.

[Michael Zolotukhin via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL269387: [LoopUnrollAnalyzer] Don't treat gep-instructions with simplified offset as… (authored by mzolotukhin).

Changed prior to commit:
  http://reviews.llvm.org/D17365?vs=48270&id=57127#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D17365

Files:
  llvm/trunk/lib/Analysis/LoopUnrollAnalyzer.cpp
  llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-cmp.ll
  llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-geps.ll

Index: llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-geps.ll
===================================================================
--- llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-geps.ll
+++ llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-geps.ll
@@ -0,0 +1,28 @@
+; RUN: opt < %s -S -loop-unroll -unroll-max-iteration-count-to-analyze=100 -unroll-dynamic-cost-savings-discount=1000 -unroll-threshold=10 -unroll-percent-dynamic-cost-saved-threshold=40 | FileCheck %s
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
+
+; When examining gep-instructions we shouldn't consider them simplified if the
+; corresponding memory access isn't simplified. Doing the opposite might bias
+; our estimate, so that we might decide to unroll even a simple memcpy loop.
+;
+; Thus, the following loop shouldn't be unrolled:
+; CHECK-LABEL: @not_simplified_geps
+; CHECK: br i1 %
+; CHECK: ret void
+define void @not_simplified_geps(i32* noalias %b, i32* noalias %c) {
+entry:
+  br label %for.body
+
+for.body:
+  %iv.0 = phi i64 [ 0, %entry ], [ %iv.1, %for.body ]
+  %arrayidx1 = getelementptr inbounds i32, i32* %b, i64 %iv.0
+  %x1 = load i32, i32* %arrayidx1, align 4
+  %arrayidx2 = getelementptr inbounds i32, i32* %c, i64 %iv.0
+  store i32 %x1, i32* %arrayidx2, align 4
+  %iv.1 = add nuw nsw i64 %iv.0, 1
+  %exitcond = icmp eq i64 %iv.1, 10
+  br i1 %exitcond, label %for.end, label %for.body
+
+for.end:
+  ret void
+}
Index: llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-cmp.ll
===================================================================
--- llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-cmp.ll
+++ llvm/trunk/test/Transforms/LoopUnroll/full-unroll-heuristics-cmp.ll
@@ -1,4 +1,4 @@
-; RUN: opt < %s -S -loop-unroll -unroll-max-iteration-count-to-analyze=100 -unroll-dynamic-cost-savings-discount=1000 -unroll-threshold=10 -unroll-percent-dynamic-cost-saved-threshold=50 | FileCheck %s
+; RUN: opt < %s -S -loop-unroll -unroll-max-iteration-count-to-analyze=100 -unroll-dynamic-cost-savings-discount=1000 -unroll-threshold=10 -unroll-percent-dynamic-cost-saved-threshold=40 | FileCheck %s
 target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
 
 @known_constant = internal unnamed_addr constant [10 x i32] [i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1], align 16
Index: llvm/trunk/lib/Analysis/LoopUnrollAnalyzer.cpp
===================================================================
--- llvm/trunk/lib/Analysis/LoopUnrollAnalyzer.cpp
+++ llvm/trunk/lib/Analysis/LoopUnrollAnalyzer.cpp
@@ -59,7 +59,7 @@
   Address.Base = Base->getValue();
   Address.Offset = Offset->getValue();
   SimplifiedAddresses[I] = Address;
-  return true;
+  return false;
 }
 
 /// Try to simplify binary operator I.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17365.57127.patch
Type: text/x-patch
Size: 2818 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160513/d6631a76/attachment.bin>