blob: 22caecf05c01aadcb800cb805137ac65a659c143 [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 Mistrih25528d62017-12-04 17:18:51 +000012; %bb.2: derived from LLVM BB %finish
13; Predecessors according to CFG: %bb.0 %bb.1
Francis Visoiu Mistriha8a83d12017-12-07 10:40:31 +000014; %0 = PHI %3, <%bb.0>, %5, <%bb.1>
15; %7 = LDIRdK 2
16; %8 = LDIRdK 1
17; CPRdRr %2, %0, implicit-def %SREG
18; BREQk <%bb.6>, implicit %SREG
Francis Visoiu Mistrih25528d62017-12-04 17:18:51 +000019; Successors according to CFG: %bb.5(?%) %bb.6(?%)
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
Francis Visoiu Mistrih25528d62017-12-04 17:18:51 +000052; CHECK: %bb.2: derived from LLVM BB %finish
Dylan McKayf7e8ec12017-12-09 07:51:43 +000053; CHECK: BREQk [[BRANCHED:%bb.[0-9]+]]
54; CHECK: RJMPk [[DIRECT:%bb.[0-9]+]]
Dylan McKay0d4778f2016-11-09 23:46:52 +000055; CHECK: Successors according to CFG
56; CHECK-SAME-DAG: {{.*}}[[BRANCHED]]
57; CHECK-SAME-DAG: {{.*}}[[DIRECT]]
Francis Visoiu Mistrih25528d62017-12-04 17:18:51 +000058; CHECK: %bb.3: derived from LLVM BB