[llvm] fd24bdb - [AArch64] Add ld3 test case for renaming in ldst-opt (NFC).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 3 11:35:16 PDT 2020


Author: Florian Hahn
Date: 2020-06-03T19:30:03+01:00
New Revision: fd24bdbe50fa4aece574d0b53e68c37d23f7df35

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

LOG: [AArch64] Add ld3 test case for renaming in ldst-opt (NFC).

The test case highlights a mis-compile reported in PR46105, where
a consecutive register list is renamed, which invalidates some other
used registers.

Added: 
    llvm/test/CodeGen/AArch64/stp-opt-with-renaming-ld3.mir

Modified: 
    

Removed: 
    


################################################################################
diff  --git a/llvm/test/CodeGen/AArch64/stp-opt-with-renaming-ld3.mir b/llvm/test/CodeGen/AArch64/stp-opt-with-renaming-ld3.mir
new file mode 100644
index 000000000000..ff1e5de46573
--- /dev/null
+++ b/llvm/test/CodeGen/AArch64/stp-opt-with-renaming-ld3.mir
@@ -0,0 +1,39 @@
+# RUN: llc -run-pass=aarch64-ldst-opt -mtriple=arm64-apple-iphoneos -aarch64-load-store-renaming=true -o - %s | FileCheck %s
+--- |
+  define void @test_ld3(<8 x i8>* %a1) {
+  entry:
+    %s1 = alloca i64
+    ret void
+  }
+
+...
+---
+# CHECK-LABEL: name: test_ld3
+# CHECK: bb.0.entry:
+# CHECK:       renamable $x0, $d1_d2_d3 = LD3Threev8b_POST killed renamable $x0, $xzr
+# CHECK-NEXT:  STPDi renamable $d0, renamable $d1, $fp, -6 :: (store 8)
+# CHECK-NEXT:  renamable $d0_d1_d2 = LD3Threev8b killed renamable $x0 :: (load 24 from %ir.a1, align 32)
+# CHECK-NEXT:  STPDi $d3, renamable $d0, $fp, -4 :: (store 8 into %ir.s1), (store 8)
+# CHECK-NEXT:  STPDi renamable $d1, renamable $d2, $fp, -2 :: (store 8)
+# CHECK-NEXT:  RET undef $lr
+#
+name:            test_ld3
+alignment:       4
+tracksRegLiveness: true
+frameInfo: {}
+machineFunctionInfo: {}
+body:             |
+  bb.0.entry:
+    liveins: $x0, $x1, $lr, $fp
+
+    renamable $x0, renamable $d0_d1_d2 = LD3Threev8b_POST killed renamable $x0, $xzr
+    STURDi renamable $d0, $fp, -48 :: (store 8)
+    STURDi renamable $d1, $fp, -40 :: (store 8)
+    STURDi renamable $d2, $fp, -32, implicit killed $d0_d1_d2 :: (store 8 into %ir.s1)
+    renamable $d0_d1_d2 = LD3Threev8b killed renamable $x0 :: (load 24 from %ir.a1, align 32)
+    STURDi renamable $d0, $fp, -24 :: (store 8)
+    STURDi renamable $d1, $fp, -16 :: (store 8)
+    STURDi renamable $d2, $fp, -8, implicit killed $d0_d1_d2 :: (store 8)
+    RET undef $lr
+
+...


        


More information about the llvm-commits mailing list