1*9880d681SAndroid Build Coastguard Worker; RUN: opt -basicaa -loop-distribute -S < %s | FileCheck %s 2*9880d681SAndroid Build Coastguard Worker 3*9880d681SAndroid Build Coastguard Worker; When emitting the memchecks for: 4*9880d681SAndroid Build Coastguard Worker; 5*9880d681SAndroid Build Coastguard Worker; for (i = 0; i < n; i++) { 6*9880d681SAndroid Build Coastguard Worker; A[i + 1] = A[i] * B[i]; 7*9880d681SAndroid Build Coastguard Worker; ======================= 8*9880d681SAndroid Build Coastguard Worker; C[i] = D[i] * E[i]; 9*9880d681SAndroid Build Coastguard Worker; } 10*9880d681SAndroid Build Coastguard Worker; 11*9880d681SAndroid Build Coastguard Worker; we had a bug when expanding the bounds for A and C. These are expanded 12*9880d681SAndroid Build Coastguard Worker; multiple times and rely on the caching in SCEV expansion to avoid any 13*9880d681SAndroid Build Coastguard Worker; redundancy. However, due to logic in SCEVExpander::ReuseOrCreateCast, we 14*9880d681SAndroid Build Coastguard Worker; can get earlier expanded values invalidated when casts are used. This test 15*9880d681SAndroid Build Coastguard Worker; ensure that we are not using the invalidated values. 16*9880d681SAndroid Build Coastguard Worker 17*9880d681SAndroid Build Coastguard Workertarget datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" 18*9880d681SAndroid Build Coastguard Worker 19*9880d681SAndroid Build Coastguard Workerdefine void @f(i32* %a1, i32* %a2, 20*9880d681SAndroid Build Coastguard Worker i32* %b, 21*9880d681SAndroid Build Coastguard Worker i32* %c1, i32* %c2, 22*9880d681SAndroid Build Coastguard Worker i32* %d, 23*9880d681SAndroid Build Coastguard Worker i32* %e) { 24*9880d681SAndroid Build Coastguard Workerentry: 25*9880d681SAndroid Build Coastguard Worker 26*9880d681SAndroid Build Coastguard Worker %cond = icmp eq i32* %e, null 27*9880d681SAndroid Build Coastguard Worker br i1 %cond, label %one, label %two 28*9880d681SAndroid Build Coastguard Workerone: 29*9880d681SAndroid Build Coastguard Worker br label %join 30*9880d681SAndroid Build Coastguard Workertwo: 31*9880d681SAndroid Build Coastguard Worker br label %join 32*9880d681SAndroid Build Coastguard Workerjoin: 33*9880d681SAndroid Build Coastguard Worker 34*9880d681SAndroid Build Coastguard Worker; The pointers need to be defined by PHIs in order for the bug to trigger. 35*9880d681SAndroid Build Coastguard Worker; Because of the PHIs the existing casts won't be at the desired location so a 36*9880d681SAndroid Build Coastguard Worker; new cast will be emitted and the old cast will get invalidated. 37*9880d681SAndroid Build Coastguard Worker; 38*9880d681SAndroid Build Coastguard Worker; These are the steps: 39*9880d681SAndroid Build Coastguard Worker; 40*9880d681SAndroid Build Coastguard Worker; 1. After the bounds for A and C are first expanded: 41*9880d681SAndroid Build Coastguard Worker; 42*9880d681SAndroid Build Coastguard Worker; join: 43*9880d681SAndroid Build Coastguard Worker; %a = phi i32* [ %a1, %one ], [ %a2, %two ] 44*9880d681SAndroid Build Coastguard Worker; %c = phi i32* [ %c1, %one ], [ %c2, %two ] 45*9880d681SAndroid Build Coastguard Worker; %c5 = bitcast i32* %c to i8* 46*9880d681SAndroid Build Coastguard Worker; %a3 = bitcast i32* %a to i8* 47*9880d681SAndroid Build Coastguard Worker; 48*9880d681SAndroid Build Coastguard Worker; 2. After A is expanded again: 49*9880d681SAndroid Build Coastguard Worker; 50*9880d681SAndroid Build Coastguard Worker; join: ; preds = %two, %one 51*9880d681SAndroid Build Coastguard Worker; %a = phi i32* [ %a1, %one ], [ %a2, %two ] 52*9880d681SAndroid Build Coastguard Worker; %c = phi i32* [ %c1, %one ], [ %c2, %two ] 53*9880d681SAndroid Build Coastguard Worker; %a3 = bitcast i32* %a to i8* <--- new 54*9880d681SAndroid Build Coastguard Worker; %c5 = bitcast i32* %c to i8* 55*9880d681SAndroid Build Coastguard Worker; %0 = bitcast i32* undef to i8* <--- old, invalidated 56*9880d681SAndroid Build Coastguard Worker; 57*9880d681SAndroid Build Coastguard Worker; 3. Finally, when C is expanded again: 58*9880d681SAndroid Build Coastguard Worker; 59*9880d681SAndroid Build Coastguard Worker; join: ; preds = %two, %one 60*9880d681SAndroid Build Coastguard Worker; %a = phi i32* [ %a1, %one ], [ %a2, %two ] 61*9880d681SAndroid Build Coastguard Worker; %c = phi i32* [ %c1, %one ], [ %c2, %two ] 62*9880d681SAndroid Build Coastguard Worker; %c5 = bitcast i32* %c to i8* <--- new 63*9880d681SAndroid Build Coastguard Worker; %a3 = bitcast i32* %a to i8* 64*9880d681SAndroid Build Coastguard Worker; %0 = bitcast i32* undef to i8* <--- old, invalidated 65*9880d681SAndroid Build Coastguard Worker; %1 = bitcast i32* undef to i8* 66*9880d681SAndroid Build Coastguard Worker 67*9880d681SAndroid Build Coastguard Worker %a = phi i32* [%a1, %one], [%a2, %two] 68*9880d681SAndroid Build Coastguard Worker %c = phi i32* [%c1, %one], [%c2, %two] 69*9880d681SAndroid Build Coastguard Worker br label %for.body 70*9880d681SAndroid Build Coastguard Worker 71*9880d681SAndroid Build Coastguard Worker 72*9880d681SAndroid Build Coastguard Worker; CHECK: [[VALUE:%[0-9a-z]+]] = bitcast i32* undef to i8* 73*9880d681SAndroid Build Coastguard Worker; CHECK-NOT: [[VALUE]] 74*9880d681SAndroid Build Coastguard Worker 75*9880d681SAndroid Build Coastguard Workerfor.body: ; preds = %for.body, %entry 76*9880d681SAndroid Build Coastguard Worker %ind = phi i64 [ 0, %join ], [ %add, %for.body ] 77*9880d681SAndroid Build Coastguard Worker 78*9880d681SAndroid Build Coastguard Worker %arrayidxA = getelementptr inbounds i32, i32* %a, i64 %ind 79*9880d681SAndroid Build Coastguard Worker %loadA = load i32, i32* %arrayidxA, align 4 80*9880d681SAndroid Build Coastguard Worker 81*9880d681SAndroid Build Coastguard Worker %arrayidxB = getelementptr inbounds i32, i32* %b, i64 %ind 82*9880d681SAndroid Build Coastguard Worker %loadB = load i32, i32* %arrayidxB, align 4 83*9880d681SAndroid Build Coastguard Worker 84*9880d681SAndroid Build Coastguard Worker %mulA = mul i32 %loadB, %loadA 85*9880d681SAndroid Build Coastguard Worker 86*9880d681SAndroid Build Coastguard Worker %add = add nuw nsw i64 %ind, 1 87*9880d681SAndroid Build Coastguard Worker %arrayidxA_plus_4 = getelementptr inbounds i32, i32* %a, i64 %add 88*9880d681SAndroid Build Coastguard Worker store i32 %mulA, i32* %arrayidxA_plus_4, align 4 89*9880d681SAndroid Build Coastguard Worker 90*9880d681SAndroid Build Coastguard Worker %arrayidxD = getelementptr inbounds i32, i32* %d, i64 %ind 91*9880d681SAndroid Build Coastguard Worker %loadD = load i32, i32* %arrayidxD, align 4 92*9880d681SAndroid Build Coastguard Worker 93*9880d681SAndroid Build Coastguard Worker %arrayidxE = getelementptr inbounds i32, i32* %e, i64 %ind 94*9880d681SAndroid Build Coastguard Worker %loadE = load i32, i32* %arrayidxE, align 4 95*9880d681SAndroid Build Coastguard Worker 96*9880d681SAndroid Build Coastguard Worker %mulC = mul i32 %loadD, %loadE 97*9880d681SAndroid Build Coastguard Worker 98*9880d681SAndroid Build Coastguard Worker %arrayidxC = getelementptr inbounds i32, i32* %c, i64 %ind 99*9880d681SAndroid Build Coastguard Worker store i32 %mulC, i32* %arrayidxC, align 4 100*9880d681SAndroid Build Coastguard Worker 101*9880d681SAndroid Build Coastguard Worker %exitcond = icmp eq i64 %add, 20 102*9880d681SAndroid Build Coastguard Worker br i1 %exitcond, label %for.end, label %for.body 103*9880d681SAndroid Build Coastguard Worker 104*9880d681SAndroid Build Coastguard Workerfor.end: ; preds = %for.body 105*9880d681SAndroid Build Coastguard Worker ret void 106*9880d681SAndroid Build Coastguard Worker} 107