[llvm] [test][AArch64][CodeGen] Delete redundant `CHECK-PIC` lines (PR #87965)
Daniil Kovalev via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 8 03:20:09 PDT 2024
https://github.com/kovdan01 updated https://github.com/llvm/llvm-project/pull/87965
>From 030c44d04a8c7dc7684f0cfbcaba7cfb16d6122c Mon Sep 17 00:00:00 2001
From: Daniil Kovalev <dkovalev at accesssoftek.com>
Date: Mon, 8 Apr 2024 10:21:52 +0300
Subject: [PATCH] [test][AArch64][CodeGen] Delete redundant check lines
llvm/test/CodeGen/AArch64/elf-globals-pic.ll:
Since https://reviews.llvm.org/D91734, elf-globals-static.ll test contains
several `CHECK-PIC` lines. They do not seem to bring any value since there
are no FileCheck run lines checking against this prefix. The right place for
such tests should be elf-globals-pic.ll, which already contains check lines
being deleted in this commit. Both elf-globals-pic.ll and
elf-globals-static.ll were created after splitting arm64-elf-globals.ll in
6dbd0eac02bf98a7585904ad69d0d9950947c9b4, and having `CHECK-PIC` lines in
elf-globals-static.ll seems like an issue occurred because of git thinking
that elf-globals-pic.ll is a new file and elf-global-static.ll is a rename
of arm64-elf-globals.ll.
llvm/test/CodeGen/AArch64/tagged-globals-pic.ll:
Similar to elf-globals-pic.ll, contains unneeded `CHECK-SELECTIONDAGISEL`
and `CHECK-GLOBALISEL` directives not checked by any FileCheck invocation.
These directives are present in tagged-globals-static.ll. Both tests are
present in the code tree since fd32639823e4d74eadbca5b0a9610a87d69b25c8
when tagged-globals.ll was splitted into tagged-globals-{pic|static}.ll.
---
.../CodeGen/AArch64/elf-globals-static.ll | 5 ----
.../CodeGen/AArch64/tagged-globals-pic.ll | 24 -------------------
2 files changed, 29 deletions(-)
diff --git a/llvm/test/CodeGen/AArch64/elf-globals-static.ll b/llvm/test/CodeGen/AArch64/elf-globals-static.ll
index 86b7c401b9a2e0..855312f8927c7d 100644
--- a/llvm/test/CodeGen/AArch64/elf-globals-static.ll
+++ b/llvm/test/CodeGen/AArch64/elf-globals-static.ll
@@ -15,11 +15,6 @@ define i8 @test_i8(i8 %new) {
; CHECK: ldrb {{w[0-9]+}}, [x[[HIREG]], :lo12:var8]
; CHECK: strb {{w[0-9]+}}, [x[[HIREG]], :lo12:var8]
-; CHECK-PIC-LABEL: test_i8:
-; CHECK-PIC: adrp x[[HIREG:[0-9]+]], :got:var8
-; CHECK-PIC: ldr x[[VAR_ADDR:[0-9]+]], [x[[HIREG]], :got_lo12:var8]
-; CHECK-PIC: ldrb {{w[0-9]+}}, [x[[VAR_ADDR]]]
-
; CHECK-FAST-LABEL: test_i8:
; CHECK-FAST: adrp x[[HIREG:[0-9]+]], var8
; CHECK-FAST: ldrb {{w[0-9]+}}, [x[[HIREG]], :lo12:var8]
diff --git a/llvm/test/CodeGen/AArch64/tagged-globals-pic.ll b/llvm/test/CodeGen/AArch64/tagged-globals-pic.ll
index 6ee0dd193c3dc4..3c4274abdd5636 100644
--- a/llvm/test/CodeGen/AArch64/tagged-globals-pic.ll
+++ b/llvm/test/CodeGen/AArch64/tagged-globals-pic.ll
@@ -31,18 +31,6 @@ define ptr @global_addr() #0 {
}
define i32 @global_load() #0 {
- ; CHECK-SELECTIONDAGISEL: global_load:
- ; CHECK-SELECTIONDAGISEL: adrp [[REG:x[0-9]+]], :pg_hi21_nc:global
- ; CHECK-SELECTIONDAGISEL: ldr w0, [[[REG]], :lo12:global]
- ; CHECK-SELECTIONDAGISEL: ret
-
- ; CHECK-GLOBALISEL: global_load:
- ; CHECK-GLOBALISEL: adrp [[REG:x[0-9]+]], :pg_hi21_nc:global
- ; CHECK-GLOBALISEL: movk [[REG]], #:prel_g3:global+4294967296
- ; CHECK-GLOBALISEL: add [[REG]], [[REG]], :lo12:global
- ; CHECK-GLOBALISEL: ldr w0, [[[REG]]]
- ; CHECK-GLOBALISEL: ret
-
; CHECK-PIC: global_load:
; CHECK-PIC: adrp [[REG:x[0-9]+]], :got:global
; CHECK-PIC: ldr [[REG]], [[[REG]], :got_lo12:global]
@@ -54,18 +42,6 @@ define i32 @global_load() #0 {
}
define void @global_store() #0 {
- ; CHECK-SELECTIONDAGISEL: global_store:
- ; CHECK-SELECTIONDAGISEL: adrp [[REG:x[0-9]+]], :pg_hi21_nc:global
- ; CHECK-SELECTIONDAGISEL: str wzr, [[[REG]], :lo12:global]
- ; CHECK-SELECTIONDAGISEL: ret
-
- ; CHECK-GLOBALISEL: global_store:
- ; CHECK-GLOBALISEL: adrp [[REG:x[0-9]+]], :pg_hi21_nc:global
- ; CHECK-GLOBALISEL: movk [[REG]], #:prel_g3:global+4294967296
- ; CHECK-GLOBALISEL: add [[REG]], [[REG]], :lo12:global
- ; CHECK-GLOBALISEL: str wzr, [[[REG]]]
- ; CHECK-GLOBALISEL: ret
-
; CHECK-PIC: global_store:
; CHECK-PIC: adrp [[REG:x[0-9]+]], :got:global
; CHECK-PIC: ldr [[REG]], [[[REG]], :got_lo12:global]
More information about the llvm-commits
mailing list