diff options
| author | Karl-Johan Karlsson <karl-johan.karlsson@ericsson.com> | 2018-06-07 07:20:33 +0000 |
|---|---|---|
| committer | Karl-Johan Karlsson <karl-johan.karlsson@ericsson.com> | 2018-06-07 07:20:33 +0000 |
| commit | abb11f805f3a2297a3968c5daf625cd882ca0fb3 (patch) | |
| tree | 9e962937f5f9cb74a4028c52f9de3be7ec50ec5f /llvm/test/CodeGen | |
| parent | 203cdee0ecac826499245aa407b98b7a355ab0c0 (diff) | |
| download | bcm5719-llvm-abb11f805f3a2297a3968c5daf625cd882ca0fb3.tar.gz bcm5719-llvm-abb11f805f3a2297a3968c5daf625cd882ca0fb3.zip | |
[BranchFolding] Fix live-in's when hoisting code
Summary:
When the branch folder hoist code into a predecessor it adjust live-in's
in the blocks it hoist code from. However it fail to handle hoisted code
that contain a defed register that originally is live-in in the block
through a super register.
This is fixed by replacing the live-in handling code with calls to
utility functions in LivePhysRegs.
Reviewers: kparzysz, gberry, MatzeB, uweigand, aprantl
Reviewed By: kparzysz
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D47529
llvm-svn: 334163
Diffstat (limited to 'llvm/test/CodeGen')
| -rw-r--r-- | llvm/test/CodeGen/SystemZ/branch-folder-hoist-livein.mir | 57 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/tail-call-conditional.mir | 4 |
2 files changed, 59 insertions, 2 deletions
diff --git a/llvm/test/CodeGen/SystemZ/branch-folder-hoist-livein.mir b/llvm/test/CodeGen/SystemZ/branch-folder-hoist-livein.mir new file mode 100644 index 00000000000..96a44768c6c --- /dev/null +++ b/llvm/test/CodeGen/SystemZ/branch-folder-hoist-livein.mir @@ -0,0 +1,57 @@ +# RUN: llc -verify-machineinstrs -O1 -mtriple=s390x-ibm-linux -o - %s -run-pass=branch-folder | FileCheck %s +--- | + target datalayout = "E-m:e-i1:8:16-i8:8:16-i64:64-f128:64-a:8:16-n32:64" + target triple = "s390x-ibm-linux" + + @b = external global i16, align 1 + @d = external global i16, align 1 + + define void @f1() { + ret void + } + +... +--- +name: f1 +tracksRegLiveness: true +body: | + bb.0: + successors: %bb.2(0x7fffffff), %bb.1(0x00000001) + liveins: + + renamable $r1d = LGRL @b :: (load 4 from got, align 8) + renamable $r1l = LH killed renamable $r1d, 0, $noreg, implicit-def $r1d :: (dereferenceable load 1 from @b) + renamable $r2l = LHI 0 + renamable $r3d = LGRL @d :: (load 4 from got, align 8) + renamable $r4d = LLILL 0, implicit-def $r4q + + CHI killed renamable $r2l, 0, implicit-def $cc + BRC 14, 6, %bb.2, implicit killed $cc + J %bb.1 + + bb.1: + successors: + liveins: $r1l, $r3d, $r4q + + renamable $r4d = COPY killed renamable $r4d, implicit killed $r4q + STH renamable $r1l, killed renamable $r3d, 0, $noreg, implicit killed $r4d :: (store 1 into @d) + + bb.2: + liveins: $r1l, $r3d, $r4q + + renamable $r4d = COPY killed renamable $r4d, implicit killed $r4q + STH renamable $r1l, killed renamable $r3d, 0, $noreg, implicit killed $r4d :: (store 1 into @d) + Return + +... + +# CHECK: renamable $r4d = COPY killed renamable $r4d, implicit killed $r4q +# CHECK-NEXT: CHI killed renamable $r2l, 0, implicit-def $cc +# CHECK-NEXT: BRC 14, 6, %bb.2, implicit killed $cc +# CHECK-NEXT: {{^ $}} +# CHECK-NEXT: bb.1: +# CHECK-NEXT: successors: +# CHECK-NEXT: liveins: $r1l, $r3d, $r4d + +# CHECK: bb.2: +# CHECK-NEXT: liveins: $r1l, $r3d, $r4d diff --git a/llvm/test/CodeGen/X86/tail-call-conditional.mir b/llvm/test/CodeGen/X86/tail-call-conditional.mir index 396fbf3bac8..47ae83dad7f 100644 --- a/llvm/test/CodeGen/X86/tail-call-conditional.mir +++ b/llvm/test/CodeGen/X86/tail-call-conditional.mir @@ -48,7 +48,7 @@ body: | ; CHECK-NEXT: $rdi = COPY $rsi ; CHECK-NEXT: $rsi = COPY $rax ; CHECK-NEXT: CMP64ri8 $rax, 9, implicit-def $eflags - ; CHECK-NEXT: TCRETURNdi64cc @f1, 0, 3, csr_64, implicit $rsp, implicit $eflags, implicit $ssp, implicit $rsp, implicit $rdi, implicit $rsi, implicit $rax, implicit-def $rax, implicit $hsi, implicit-def $hsi, implicit $sil, implicit-def $sil, implicit $si, implicit-def $si, implicit $esi, implicit-def $esi, implicit $rsi, implicit-def $rsi, implicit $hdi, implicit-def $hdi, implicit $dil, implicit-def $dil, implicit $di, implicit-def $di, implicit $edi, implicit-def $edi, implicit $rdi, implicit-def $rdi, implicit $hax, implicit-def $hax, implicit $ah, implicit-def $ah, implicit $al, implicit-def $al, implicit $ax, implicit-def $ax, implicit $eax, implicit-def $eax + ; CHECK-NEXT: TCRETURNdi64cc @f1, 0, 3, csr_64, implicit $rsp, implicit $eflags, implicit $ssp, implicit $rsp, implicit $rdi, implicit $rsi, implicit $rdi, implicit-def $rdi, implicit $hsi, implicit-def $hsi, implicit $sil, implicit-def $sil, implicit $si, implicit-def $si, implicit $esi, implicit-def $esi, implicit $rsi, implicit-def $rsi, implicit $hdi, implicit-def $hdi, implicit $dil, implicit-def $dil, implicit $di, implicit-def $di, implicit $edi, implicit-def $edi bb.1: successors: %bb.2, %bb.3 @@ -67,7 +67,7 @@ body: | TCRETURNdi64 @f1, 0, csr_64, implicit $rsp, implicit $rdi, implicit $rsi ; CHECK: bb.2: - ; CHECK-NEXT: liveins: $rax, $rdi, $rsi + ; CHECK-NEXT: liveins: $rdi, $rsi ; CHECK-NEXT: {{^ $}} ; CHECK-NEXT: TCRETURNdi64 @f2, 0, csr_64, implicit $rsp, implicit $rdi, implicit $rsi |

