xref: /aosp_15_r20/external/llvm/test/Transforms/LoopDistribute/bounds-expansion-bug.ll (revision 9880d6810fe72a1726cb53787c6711e909410d58)
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