diff options
author | Adam Nemet <anemet@apple.com> | 2015-03-10 17:40:43 +0000 |
---|---|---|
committer | Adam Nemet <anemet@apple.com> | 2015-03-10 17:40:43 +0000 |
commit | 8e7d56f6f7a7ba4e872675dd30c2fa689f02221d (patch) | |
tree | 9ad3777fe4480b4e835e97867a241645e89df0d4 /test | |
parent | 7063c7e0844bbd2e59ef639dd4b88907b2520aa9 (diff) |
[LoopAccesses 3/3] Print the dependences with -analyze
The dependences are now expose through the new getInterestingDependences
API so we can use that with -analyze too and fix the FIXME.
This lets us remove the test that relied on -debug to check the
dependences.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@231807 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test')
-rw-r--r-- | test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks-no-dbg.ll | 60 | ||||
-rw-r--r-- | test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll | 10 |
2 files changed, 4 insertions, 66 deletions
diff --git a/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks-no-dbg.ll b/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks-no-dbg.ll deleted file mode 100644 index 6770f9253a0..00000000000 --- a/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks-no-dbg.ll +++ /dev/null @@ -1,60 +0,0 @@ -; RUN: opt -loop-accesses -analyze < %s | FileCheck %s - -; FIXME: This is the non-debug version of unsafe-and-rt-checks.ll not -; requiring "asserts". Once we can check memory dependences without -debug, -; we should remove this test. - -; Analyze this loop: -; for (i = 0; i < n; i++) -; A[i + 1] = A[i] * B[i] * C[i]; - -target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" -target triple = "x86_64-apple-macosx10.10.0" - -; CHECK: Report: unsafe dependent memory operations in loop - -; CHECK: Run-time memory checks: -; CHECK-NEXT: 0: -; CHECK-NEXT: %arrayidxA_plus_2 = getelementptr inbounds i16, i16* %a, i64 %add -; CHECK-NEXT: %arrayidxB = getelementptr inbounds i16, i16* %b, i64 %storemerge3 -; CHECK-NEXT: 1: -; CHECK-NEXT: %arrayidxA_plus_2 = getelementptr inbounds i16, i16* %a, i64 %add -; CHECK-NEXT: %arrayidxC = getelementptr inbounds i16, i16* %c, i64 %storemerge3 - -@n = global i32 20, align 4 -@B = common global i16* null, align 8 -@A = common global i16* null, align 8 -@C = common global i16* null, align 8 - -define void @f() { -entry: - %a = load i16*, i16** @A, align 8 - %b = load i16*, i16** @B, align 8 - %c = load i16*, i16** @C, align 8 - br label %for.body - -for.body: ; preds = %for.body, %entry - %storemerge3 = phi i64 [ 0, %entry ], [ %add, %for.body ] - - %arrayidxA = getelementptr inbounds i16, i16* %a, i64 %storemerge3 - %loadA = load i16, i16* %arrayidxA, align 2 - - %arrayidxB = getelementptr inbounds i16, i16* %b, i64 %storemerge3 - %loadB = load i16, i16* %arrayidxB, align 2 - - %arrayidxC = getelementptr inbounds i16, i16* %c, i64 %storemerge3 - %loadC = load i16, i16* %arrayidxC, align 2 - - %mul = mul i16 %loadB, %loadA - %mul1 = mul i16 %mul, %loadC - - %add = add nuw nsw i64 %storemerge3, 1 - %arrayidxA_plus_2 = getelementptr inbounds i16, i16* %a, i64 %add - store i16 %mul1, i16* %arrayidxA_plus_2, align 2 - - %exitcond = icmp eq i64 %add, 20 - br i1 %exitcond, label %for.end, label %for.body - -for.end: ; preds = %for.body - ret void -} diff --git a/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll b/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll index a7a324bd9a0..a11fd7f3bff 100644 --- a/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll +++ b/test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll @@ -1,6 +1,4 @@ ; RUN: opt -loop-accesses -analyze < %s | FileCheck %s -; RUN: opt -loop-accesses -analyze -debug-only=loop-accesses < %s 2>&1 | FileCheck %s --check-prefix=DEBUG -; REQUIRES: asserts ; Analyze this loop: ; for (i = 0; i < n; i++) @@ -10,10 +8,10 @@ target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" target triple = "x86_64-apple-macosx10.10.0" ; CHECK: Report: unsafe dependent memory operations in loop - -; DEBUG: LAA: Distance for %loadA = load i16, i16* %arrayidxA, align 2 to store i16 %mul1, i16* %arrayidxA_plus_2, align 2: 2 -; DEBUG-NEXT: LAA: Failure because of Positive distance 2 - +; CHECK-NEXT: Interesting Dependences: +; CHECK-NEXT: Backward: +; CHECK-NEXT: %loadA = load i16, i16* %arrayidxA, align 2 -> +; CHECK-NEXT: store i16 %mul1, i16* %arrayidxA_plus_2, align 2 ; CHECK: Run-time memory checks: ; CHECK-NEXT: 0: ; CHECK-NEXT: %arrayidxA_plus_2 = getelementptr inbounds i16, i16* %a, i64 %add |