[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 00:32:56 PDT 2024


https://github.com/kovdan01 created https://github.com/llvm/llvm-project/pull/87965

Since https://reviews.llvm.org/D91734, CodeGen/AArch64/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

>From 72c2cb1af931cdff42ce54298963f4f8c889ce40 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-PIC` lines

Since https://reviews.llvm.org/D91734, CodeGen/AArch64/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/elf-globals-static.ll | 5 -----
 1 file changed, 5 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]



More information about the llvm-commits mailing list