diff options
| author | Quentin Colombet <qcolombet@apple.com> | 2017-07-07 19:25:45 +0000 |
|---|---|---|
| committer | Quentin Colombet <qcolombet@apple.com> | 2017-07-07 19:25:45 +0000 |
| commit | 4e13bac06cfa8298baf441282a56e3c608535445 (patch) | |
| tree | 2558faaf5e8bf4cc274a8590ba85b00be1e4a115 /test/CodeGen/SystemZ | |
| parent | b8caa0933a1e5ab2bd0668caf4b17d2c9a2aa72c (diff) | |
[RegAllocFast] Don't insert kill flags of super-register for partial kill
When reusing a register for a new definition, the fast register allocator
used to insert a kill flag at the previous last use of that register to
inform later passes that this register is free between the redef and the
last use. However, this may be wrong when subregisters are involved.
Indeed, a partially redef would have trigger a kill of the full super
register, potentially wrongly marking all the other subregisters as
free. Given we don't track which lanes are still live, we cannot set the
kill flag in such case.
Note: This bug has been latent for about 7 years (r104056).
llvmg.org/PR33677
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@307428 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGen/SystemZ')
| -rw-r--r-- | test/CodeGen/SystemZ/regalloc-fast-invalid-kill-flag.mir | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/test/CodeGen/SystemZ/regalloc-fast-invalid-kill-flag.mir b/test/CodeGen/SystemZ/regalloc-fast-invalid-kill-flag.mir new file mode 100644 index 00000000000..8798fcecfc3 --- /dev/null +++ b/test/CodeGen/SystemZ/regalloc-fast-invalid-kill-flag.mir @@ -0,0 +1,34 @@ +# RUN: llc -verify-machineinstrs -run-pass regallocfast -mtriple s390x-ibm-linux -o - %s | FileCheck %s +--- | + + @g_167 = external global [5 x i64], align 8 + define void @main() local_unnamed_addr { + ret void + } +... +# Make sure the usage of different subregisters on the same virtual register +# does not result in invalid kill flags. +# PR33677 +--- +name: main +alignment: 2 +tracksRegLiveness: true +registers: + - { id: 0, class: gr128bit } + - { id: 1, class: gr64bit } + - { id: 2, class: addr64bit } +# CHECK: %r0q = L128 +# CHECK-NEXT: %r0l = COPY %r1l +# Although R0L partially redefines R0Q, it must not mark R0Q as kill +# because R1D is still live through that instruction. +# CHECK-NOT: %r0q<imp-use,kill> +# CHECK-NEXT: %r2d = COPY %r1d +# CHECK-NEXT: LARL +body: | + bb.0: + %0.subreg_hl32 = COPY %0.subreg_l32 + %1 = COPY %0.subreg_l64 + %2 = LARL @g_167 + STC %1.subreg_l32, %2, 8, _ + +... |