blob: 08382c92df5d3edea586518b3a1c1c1bc15827e1 [file] [log] [blame]
Jun Bum Lim90b6b502016-12-16 20:38:39 +00001; RUN: llc -march=avr -print-after=expand-isel-pseudos -cgp-freq-ratio-to-skip-merge=10 < %s 2>&1 | FileCheck %s
Dylan McKay0d4778f2016-11-09 23:46:52 +00002
3; Because `switch` seems to trigger Machine Basic Blocks to be ordered
4; in a different order than they were constructed, this exposes an
5; error in the `expand-isel-pseudos` pass. Specifically, it thought we
6; could always fallthrough to a newly-constructed MBB. However,
7; there's no guarantee that either of the constructed MBBs need to
8; occur immediately after the currently-focused one!
9;
10; This issue manifests in a CFG that looks something like this:
11;
Francis Visoiu Mistrihda89d182018-02-08 05:02:00 +000012; %bb.2.finish:
Francis Visoiu Mistrih39ec2e92018-02-09 00:10:31 +000013; successors: %bb.5(?%) %bb.6(?%)
Francis Visoiu Mistrih25528d62017-12-04 17:18:51 +000014; Predecessors according to CFG: %bb.0 %bb.1
Francis Visoiu Mistriha8a83d12017-12-07 10:40:31 +000015; %0 = PHI %3, <%bb.0>, %5, <%bb.1>
16; %7 = LDIRdK 2
17; %8 = LDIRdK 1
18; CPRdRr %2, %0, implicit-def %SREG
19; BREQk <%bb.6>, implicit %SREG
Dylan McKay0d4778f2016-11-09 23:46:52 +000020;
Francis Visoiu Mistrih25528d62017-12-04 17:18:51 +000021; The code assumes it the fallthrough block after this is %bb.5, but
22; it's actually %bb.3! To be proper, there should be an unconditional
23; jump tying this block to %bb.5.
Dylan McKay0d4778f2016-11-09 23:46:52 +000024
25define i8 @select_must_add_unconditional_jump(i8 %arg0, i8 %arg1) unnamed_addr {
26entry-block:
27 switch i8 %arg0, label %dead [
28 i8 0, label %zero
29 i8 1, label %one
30 ]
31
32zero:
33 br label %finish
34
35one:
36 br label %finish
37
38finish:
39 %predicate = phi i8 [ 50, %zero ], [ 100, %one ]
40 %is_eq = icmp eq i8 %arg1, %predicate
41 %result = select i1 %is_eq, i8 1, i8 2
42 ret i8 %result
43
44dead:
45 ret i8 0
46}
47
48; This check may be a bit brittle, but the important thing is that the
49; basic block containing `select` needs to contain explicit jumps to
50; both successors.
51
Dylan McKay820553f2018-02-08 09:17:11 +000052; CHECK: bb.2.finish:
Francis Visoiu Mistrih39ec2e92018-02-09 00:10:31 +000053; CHECK: successors:
Dylan McKayf7e8ec12017-12-09 07:51:43 +000054; CHECK: BREQk [[BRANCHED:%bb.[0-9]+]]
55; CHECK: RJMPk [[DIRECT:%bb.[0-9]+]]
Dylan McKay0d4778f2016-11-09 23:46:52 +000056; CHECK-SAME-DAG: {{.*}}[[BRANCHED]]
57; CHECK-SAME-DAG: {{.*}}[[DIRECT]]
Dylan McKay820553f2018-02-08 09:17:11 +000058; CHECK: bb.3.dead: