[llvm] 3abd0ba - [NFC][RISCV] Add test showing wrong stack slot for GPR and RVV spilled registers

Roger Ferrer Ibanez via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 10:03:35 PDT 2021


Author: Roger Ferrer Ibanez
Date: 2021-03-29T17:03:18Z
New Revision: 3abd0bacc2c53712a175d3d2dbc75a30f0b7fd3d

URL: https://github.com/llvm/llvm-project/commit/3abd0bacc2c53712a175d3d2dbc75a30f0b7fd3d
DIFF: https://github.com/llvm/llvm-project/commit/3abd0bacc2c53712a175d3d2dbc75a30f0b7fd3d.diff

LOG: [NFC][RISCV] Add test showing wrong stack slot for GPR and RVV spilled registers

This testcase shows that we attempt to assign the same offset sp + 16 to
two different stack objects.

The fix will come in a later change.

Differential Revision: https://reviews.llvm.org/D98801

Added: 
    llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv32.mir
    llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv64.mir

Modified: 
    

Removed: 
    


################################################################################
diff  --git a/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv32.mir b/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv32.mir
new file mode 100644
index 000000000000..65cd40304288
--- /dev/null
+++ b/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv32.mir
@@ -0,0 +1,49 @@
+# NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+# RUN: llc -mtriple=riscv32 -mattr=+m,+experimental-v  -o - %s \
+# RUN:   -start-before=prologepilog | FileCheck %s
+#
+# This test checks that we are assigning the right stack slot to GPRs and to
+# vector registers (VRs). If this test changes, make sure there is no overlap
+# between slots for GPRs and VRs.
+--- |
+  define void @foo() #0 {
+  ; CHECK-LABEL: foo:
+  ; CHECK:       # %bb.0: # %entry
+  ; CHECK-NEXT:    addi sp, sp, -16
+  ; CHECK-NEXT:    sw s9, 12(sp) # 4-byte Folded Spill
+  ; CHECK-NEXT:    csrr a1, vlenb
+  ; CHECK-NEXT:    slli a1, a1, 1
+  ; CHECK-NEXT:    sub sp, sp, a1
+  ; CHECK-NEXT:    sw a0, 8(sp) # 4-byte Folded Spill
+  ; CHECK-NEXT:    vs2r.v v30, (sp) # Unknown-size Folded Spill
+  ; CHECK-NEXT:    csrr a0, vlenb
+  ; CHECK-NEXT:    slli a0, a0, 1
+  ; CHECK-NEXT:    add sp, sp, a0
+  ; CHECK-NEXT:    lw s9, 12(sp) # 4-byte Folded Reload
+  ; CHECK-NEXT:    addi sp, sp, 16
+  ; CHECK-NEXT:    ret
+  entry:
+    ret void
+  }
+
+  attributes #0 = { nounwind }
+...
+---
+name:            foo
+alignment:       2
+frameInfo:
+  maxAlignment:    8
+stack:
+  - { id: 0, type: spill-slot, size: 4, alignment: 4 }
+  - { id: 1, type: spill-slot, size: 16, alignment: 8, stack-id: scalable-vector }
+machineFunctionInfo: {}
+body:             |
+  bb.0.entry:
+    liveins: $x10, $v30m2
+
+    $x25 = COPY $x10
+    SW renamable $x25, %stack.0, 0 :: (store 4 into %stack.0)
+    PseudoVSPILL_M2 renamable $v30m2, %stack.1 :: (store unknown-size into %stack.1, align 8)
+    PseudoRET
+
+...

diff  --git a/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv64.mir b/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv64.mir
new file mode 100644
index 000000000000..dc090c51d45f
--- /dev/null
+++ b/llvm/test/CodeGen/RISCV/rvv/wrong-stack-slot-rv64.mir
@@ -0,0 +1,51 @@
+# NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+# RUN: llc -mtriple=riscv64 -mattr=+m,+experimental-v  -o - %s \
+# RUN:   -start-before=prologepilog | FileCheck %s
+#
+# This test checks that we are assigning the right stack slot to GPRs and to
+# vector registers (VRs). If this test changes, make sure there is no overlap
+# between slots for GPRs and VRs.
+--- |
+  define void @foo() #0 {
+  ; CHECK-LABEL: foo:
+  ; CHECK:       # %bb.0: # %entry
+  ; CHECK-NEXT:    addi sp, sp, -32
+  ; CHECK-NEXT:    sd s9, 24(sp) # 8-byte Folded Spill
+  ; CHECK-NEXT:    csrr a1, vlenb
+  ; CHECK-NEXT:    slli a1, a1, 1
+  ; CHECK-NEXT:    sub sp, sp, a1
+  ; CHECK-NEXT:    sd a0, 16(sp) # 8-byte Folded Spill
+  ; CHECK-NEXT:    addi a0, sp, 16
+  ; CHECK-NEXT:    vs2r.v v30, (a0) # Unknown-size Folded Spill
+  ; CHECK-NEXT:    csrr a0, vlenb
+  ; CHECK-NEXT:    slli a0, a0, 1
+  ; CHECK-NEXT:    add sp, sp, a0
+  ; CHECK-NEXT:    ld s9, 24(sp) # 8-byte Folded Reload
+  ; CHECK-NEXT:    addi sp, sp, 32
+  ; CHECK-NEXT:    ret
+  entry:
+    ret void
+  }
+
+  attributes #0 = { nounwind }
+...
+---
+name:            foo
+alignment:       2
+tracksRegLiveness: true
+frameInfo:
+  maxAlignment:    8
+stack:
+  - { id: 0, type: spill-slot, size: 8, alignment: 8 }
+  - { id: 1, type: spill-slot, size: 16, alignment: 8, stack-id: scalable-vector }
+machineFunctionInfo: {}
+body:             |
+  bb.0.entry:
+    liveins: $x10, $v30m2
+
+    $x25 = COPY $x10
+    SD renamable $x25, %stack.0, 0 :: (store 8 into %stack.0)
+    PseudoVSPILL_M2 renamable $v30m2, %stack.1 :: (store unknown-size into %stack.1, align 8)
+    PseudoRET
+
+...


        


More information about the llvm-commits mailing list