1*9880d681SAndroid Build Coastguard Worker; RUN: opt -loop-reduce -S < %s | FileCheck %s 2*9880d681SAndroid Build Coastguard Worker; 3*9880d681SAndroid Build Coastguard Worker; <rdar://10619599> "SelectionDAGBuilder shouldn't visit PHI nodes!" assert. 4*9880d681SAndroid Build Coastguard Worker; <rdar://10655343> SCEVExpander segfault on simple test case 5*9880d681SAndroid Build Coastguard Worker 6*9880d681SAndroid Build Coastguard Workertarget datalayout = "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64-f80:128:128-f128:128:128-n8:16:32" 7*9880d681SAndroid Build Coastguard Workertarget triple = "i386-apple-darwin" 8*9880d681SAndroid Build Coastguard Worker 9*9880d681SAndroid Build Coastguard Worker; LSR should convert the inner loop (bb7.us) IV (j.01.us) into float*. 10*9880d681SAndroid Build Coastguard Worker; This involves a nested AddRec, the outer AddRec's loop invariant components 11*9880d681SAndroid Build Coastguard Worker; cannot find a preheader, so they should be expanded in the loop header 12*9880d681SAndroid Build Coastguard Worker; (bb7.lr.ph.us) below the existing phi i.12.us. 13*9880d681SAndroid Build Coastguard Worker; Currently, LSR won't kick in on such loops. 14*9880d681SAndroid Build Coastguard Worker; CHECK-LABEL: @nopreheader( 15*9880d681SAndroid Build Coastguard Worker; CHECK: bb7.us: 16*9880d681SAndroid Build Coastguard Worker; CHECK-NOT: phi float* 17*9880d681SAndroid Build Coastguard Worker; CHECK: %j.01.us = phi i32 18*9880d681SAndroid Build Coastguard Worker; CHECK-NOT: phi float* 19*9880d681SAndroid Build Coastguard Workerdefine void @nopreheader(float* nocapture %a, i32 %n) nounwind { 20*9880d681SAndroid Build Coastguard Workerentry: 21*9880d681SAndroid Build Coastguard Worker %0 = sdiv i32 %n, undef 22*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb10.preheader] 23*9880d681SAndroid Build Coastguard Worker 24*9880d681SAndroid Build Coastguard Workerbb10.preheader: ; preds = %bb4 25*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb8.preheader.lr.ph, label %return] 26*9880d681SAndroid Build Coastguard Worker 27*9880d681SAndroid Build Coastguard Workerbb8.preheader.lr.ph: ; preds = %bb10.preheader 28*9880d681SAndroid Build Coastguard Worker indirectbr i8* null, [label %bb7.lr.ph.us, label %bb9] 29*9880d681SAndroid Build Coastguard Worker 30*9880d681SAndroid Build Coastguard Workerbb7.lr.ph.us: ; preds = %bb9.us, %bb8.preheader.lr.ph 31*9880d681SAndroid Build Coastguard Worker %i.12.us = phi i32 [ %2, %bb9.us ], [ 0, %bb8.preheader.lr.ph ] 32*9880d681SAndroid Build Coastguard Worker %tmp30 = mul i32 %0, %i.12.us 33*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb7.us] 34*9880d681SAndroid Build Coastguard Worker 35*9880d681SAndroid Build Coastguard Workerbb7.us: ; preds = %bb7.lr.ph.us, %bb7.us 36*9880d681SAndroid Build Coastguard Worker %j.01.us = phi i32 [ 0, %bb7.lr.ph.us ], [ %1, %bb7.us ] 37*9880d681SAndroid Build Coastguard Worker %tmp31 = add i32 %tmp30, %j.01.us 38*9880d681SAndroid Build Coastguard Worker %scevgep9 = getelementptr float, float* %a, i32 %tmp31 39*9880d681SAndroid Build Coastguard Worker store float undef, float* %scevgep9, align 1 40*9880d681SAndroid Build Coastguard Worker %1 = add nsw i32 %j.01.us, 1 41*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb9.us, label %bb7.us] 42*9880d681SAndroid Build Coastguard Worker 43*9880d681SAndroid Build Coastguard Workerbb9.us: ; preds = %bb7.us 44*9880d681SAndroid Build Coastguard Worker %2 = add nsw i32 %i.12.us, 1 45*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb7.lr.ph.us, label %return] 46*9880d681SAndroid Build Coastguard Worker 47*9880d681SAndroid Build Coastguard Workerbb9: ; preds = %bb9, %bb8.preheader.lr.ph 48*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb9, label %return] 49*9880d681SAndroid Build Coastguard Worker 50*9880d681SAndroid Build Coastguard Workerreturn: ; preds = %bb9, %bb9.us, %bb10.preheader 51*9880d681SAndroid Build Coastguard Worker ret void 52*9880d681SAndroid Build Coastguard Worker} 53*9880d681SAndroid Build Coastguard Worker 54*9880d681SAndroid Build Coastguard Worker; In this case, SCEVExpander simply cannot materialize the AddRecExpr 55*9880d681SAndroid Build Coastguard Worker; that LSR picks. We must detect that %bb8.preheader does not have a 56*9880d681SAndroid Build Coastguard Worker; preheader and avoid performing LSR on %bb7. 57*9880d681SAndroid Build Coastguard Worker; CHECK-LABEL: @nopreheader2( 58*9880d681SAndroid Build Coastguard Worker; CHECK: bb7: 59*9880d681SAndroid Build Coastguard Worker; CHECK: %indvar = phi i32 60*9880d681SAndroid Build Coastguard Workerdefine fastcc void @nopreheader2([200 x i32]* nocapture %Array2) nounwind { 61*9880d681SAndroid Build Coastguard Workerentry: 62*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb] 63*9880d681SAndroid Build Coastguard Worker 64*9880d681SAndroid Build Coastguard Workerbb: ; preds = %bb, %entry 65*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb3, label %bb] 66*9880d681SAndroid Build Coastguard Worker 67*9880d681SAndroid Build Coastguard Workerbb3: ; preds = %bb3, %bb 68*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb8.preheader, label %bb3] 69*9880d681SAndroid Build Coastguard Worker 70*9880d681SAndroid Build Coastguard Workerbb8.preheader: ; preds = %bb9, %bb3 71*9880d681SAndroid Build Coastguard Worker %indvar5 = phi i32 [ %indvar.next6, %bb9 ], [ 0, %bb3 ] 72*9880d681SAndroid Build Coastguard Worker %tmp26 = add i32 %indvar5, 13 73*9880d681SAndroid Build Coastguard Worker indirectbr i8* null, [label %bb7] 74*9880d681SAndroid Build Coastguard Worker 75*9880d681SAndroid Build Coastguard Workerbb7: ; preds = %bb8.preheader, %bb7 76*9880d681SAndroid Build Coastguard Worker %indvar = phi i32 [ 0, %bb8.preheader ], [ %indvar.next, %bb7 ] 77*9880d681SAndroid Build Coastguard Worker %scevgep = getelementptr [200 x i32], [200 x i32]* %Array2, i32 %tmp26, i32 %indvar 78*9880d681SAndroid Build Coastguard Worker store i32 undef, i32* %scevgep, align 4 79*9880d681SAndroid Build Coastguard Worker %indvar.next = add i32 %indvar, 1 80*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %bb9, label %bb7] 81*9880d681SAndroid Build Coastguard Worker 82*9880d681SAndroid Build Coastguard Workerbb9: ; preds = %bb7 83*9880d681SAndroid Build Coastguard Worker %indvar.next6 = add i32 %indvar5, 1 84*9880d681SAndroid Build Coastguard Worker indirectbr i8* undef, [label %return, label %bb8.preheader] 85*9880d681SAndroid Build Coastguard Worker 86*9880d681SAndroid Build Coastguard Workerreturn: ; preds = %bb9 87*9880d681SAndroid Build Coastguard Worker ret void 88*9880d681SAndroid Build Coastguard Worker} 89