1*9880d681SAndroid Build Coastguard Worker; RUN: opt -basicaa -loop-accesses -analyze < %s | FileCheck %s 2*9880d681SAndroid Build Coastguard Worker; RUN: opt -passes='require<scalar-evolution>,require<aa>,loop(print-access-info)' -disable-output < %s 2>&1 | FileCheck %s 3*9880d681SAndroid Build Coastguard Worker 4*9880d681SAndroid Build Coastguard Worker; This loop: 5*9880d681SAndroid Build Coastguard Worker; 6*9880d681SAndroid Build Coastguard Worker; int **A; 7*9880d681SAndroid Build Coastguard Worker; for (i) 8*9880d681SAndroid Build Coastguard Worker; for (j) { 9*9880d681SAndroid Build Coastguard Worker; A[i][j] = A[i-1][j] * B[j] 10*9880d681SAndroid Build Coastguard Worker; B[j+1] = 2 // backward dep between this and the previous 11*9880d681SAndroid Build Coastguard Worker; } 12*9880d681SAndroid Build Coastguard Worker; 13*9880d681SAndroid Build Coastguard Worker; is transformed by Load-PRE to stash away A[i] for the next iteration of the 14*9880d681SAndroid Build Coastguard Worker; outer loop: 15*9880d681SAndroid Build Coastguard Worker; 16*9880d681SAndroid Build Coastguard Worker; Curr = A[0]; // Prev_0 17*9880d681SAndroid Build Coastguard Worker; for (i: 1..N) { 18*9880d681SAndroid Build Coastguard Worker; Prev = Curr; // Prev = PHI (Prev_0, Curr) 19*9880d681SAndroid Build Coastguard Worker; Curr = A[i]; 20*9880d681SAndroid Build Coastguard Worker; for (j: 0..N) { 21*9880d681SAndroid Build Coastguard Worker; Curr[j] = Prev[j] * B[j] 22*9880d681SAndroid Build Coastguard Worker; B[j+1] = 2 // backward dep between this and the previous 23*9880d681SAndroid Build Coastguard Worker; } 24*9880d681SAndroid Build Coastguard Worker; } 25*9880d681SAndroid Build Coastguard Worker; 26*9880d681SAndroid Build Coastguard Worker; Since A[i] and A[i-1] are likely to be independent, getUnderlyingObjects 27*9880d681SAndroid Build Coastguard Worker; should not assume that Curr and Prev share the same underlying object. 28*9880d681SAndroid Build Coastguard Worker; 29*9880d681SAndroid Build Coastguard Worker; If it did we would try to dependence-analyze Curr and Prev and the analysis 30*9880d681SAndroid Build Coastguard Worker; would fail with non-constant distance. 31*9880d681SAndroid Build Coastguard Worker; 32*9880d681SAndroid Build Coastguard Worker; To illustrate one of the negative consequences of this, if the loop has a 33*9880d681SAndroid Build Coastguard Worker; backward dependence we won't detect this but instead fully fall back on 34*9880d681SAndroid Build Coastguard Worker; memchecks (that is what LAA does after encountering a case of non-constant 35*9880d681SAndroid Build Coastguard Worker; distance). 36*9880d681SAndroid Build Coastguard Worker 37*9880d681SAndroid Build Coastguard Workertarget datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" 38*9880d681SAndroid Build Coastguard Workertarget triple = "x86_64-apple-macosx10.10.0" 39*9880d681SAndroid Build Coastguard Worker 40*9880d681SAndroid Build Coastguard Worker; CHECK: for_j.body: 41*9880d681SAndroid Build Coastguard Worker; CHECK-NEXT: Report: unsafe dependent memory operations in loop 42*9880d681SAndroid Build Coastguard Worker; CHECK-NEXT: Dependences: 43*9880d681SAndroid Build Coastguard Worker; CHECK-NEXT: Backward: 44*9880d681SAndroid Build Coastguard Worker; CHECK-NEXT: %loadB = load i8, i8* %gepB, align 1 -> 45*9880d681SAndroid Build Coastguard Worker; CHECK-NEXT: store i8 2, i8* %gepB_plus_one, align 1 46*9880d681SAndroid Build Coastguard Worker 47*9880d681SAndroid Build Coastguard Workerdefine void @f(i8** noalias %A, i8* noalias %B, i64 %N) { 48*9880d681SAndroid Build Coastguard Workerfor_i.preheader: 49*9880d681SAndroid Build Coastguard Worker %prev_0 = load i8*, i8** %A, align 8 50*9880d681SAndroid Build Coastguard Worker br label %for_i.body 51*9880d681SAndroid Build Coastguard Worker 52*9880d681SAndroid Build Coastguard Workerfor_i.body: 53*9880d681SAndroid Build Coastguard Worker %i = phi i64 [1, %for_i.preheader], [%i.1, %for_j.end] 54*9880d681SAndroid Build Coastguard Worker %prev = phi i8* [%prev_0, %for_i.preheader], [%curr, %for_j.end] 55*9880d681SAndroid Build Coastguard Worker %gep = getelementptr inbounds i8*, i8** %A, i64 %i 56*9880d681SAndroid Build Coastguard Worker %curr = load i8*, i8** %gep, align 8 57*9880d681SAndroid Build Coastguard Worker br label %for_j.preheader 58*9880d681SAndroid Build Coastguard Worker 59*9880d681SAndroid Build Coastguard Workerfor_j.preheader: 60*9880d681SAndroid Build Coastguard Worker br label %for_j.body 61*9880d681SAndroid Build Coastguard Worker 62*9880d681SAndroid Build Coastguard Workerfor_j.body: 63*9880d681SAndroid Build Coastguard Worker %j = phi i64 [0, %for_j.preheader], [%j.1, %for_j.body] 64*9880d681SAndroid Build Coastguard Worker 65*9880d681SAndroid Build Coastguard Worker %gepPrev = getelementptr inbounds i8, i8* %prev, i64 %j 66*9880d681SAndroid Build Coastguard Worker %gepCurr = getelementptr inbounds i8, i8* %curr, i64 %j 67*9880d681SAndroid Build Coastguard Worker %gepB = getelementptr inbounds i8, i8* %B, i64 %j 68*9880d681SAndroid Build Coastguard Worker 69*9880d681SAndroid Build Coastguard Worker %loadPrev = load i8, i8* %gepPrev, align 1 70*9880d681SAndroid Build Coastguard Worker %loadB = load i8, i8* %gepB, align 1 71*9880d681SAndroid Build Coastguard Worker 72*9880d681SAndroid Build Coastguard Worker %mul = mul i8 %loadPrev, %loadB 73*9880d681SAndroid Build Coastguard Worker 74*9880d681SAndroid Build Coastguard Worker store i8 %mul, i8* %gepCurr, align 1 75*9880d681SAndroid Build Coastguard Worker 76*9880d681SAndroid Build Coastguard Worker %gepB_plus_one = getelementptr inbounds i8, i8* %gepB, i64 1 77*9880d681SAndroid Build Coastguard Worker store i8 2, i8* %gepB_plus_one, align 1 78*9880d681SAndroid Build Coastguard Worker 79*9880d681SAndroid Build Coastguard Worker %j.1 = add nuw i64 %j, 1 80*9880d681SAndroid Build Coastguard Worker %exitcondj = icmp eq i64 %j.1, %N 81*9880d681SAndroid Build Coastguard Worker br i1 %exitcondj, label %for_j.end, label %for_j.body 82*9880d681SAndroid Build Coastguard Worker 83*9880d681SAndroid Build Coastguard Workerfor_j.end: 84*9880d681SAndroid Build Coastguard Worker 85*9880d681SAndroid Build Coastguard Worker %i.1 = add nuw i64 %i, 1 86*9880d681SAndroid Build Coastguard Worker %exitcond = icmp eq i64 %i.1, %N 87*9880d681SAndroid Build Coastguard Worker br i1 %exitcond, label %for_i.end, label %for_i.body 88*9880d681SAndroid Build Coastguard Worker 89*9880d681SAndroid Build Coastguard Workerfor_i.end: 90*9880d681SAndroid Build Coastguard Worker ret void 91*9880d681SAndroid Build Coastguard Worker} 92