Skip to content

Commit a08447d

Browse files
committed
[LLD][ELF][AArch64] Update test with incorrect REQUIRES line [NFC]
D54759 introduced aarch64-combined-dynrel.s and aarch64-combined-dynrel-ifunc.s . Unfortunately the requires line at the top was AArch64 instead of aarch64 which means they were never run. Update the tests to use aarch64 and fix to match current lld output. Differential Revision: https://reviews.llvm.org/D117896
1 parent 0d1308a commit a08447d

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

lld/test/ELF/aarch64-combined-dynrel-ifunc.s

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: AArch64
1+
// REQUIRES: aarch64
22
// RUN: llvm-mc -filetype=obj -triple=aarch64-linux-gnu %p/Inputs/shared.s -o %t-lib.o
33
// RUN: llvm-mc -filetype=obj -triple=aarch64-linux-gnu %s -o %t.o
44
// RUN: ld.lld %t-lib.o --shared -o %t.so
@@ -42,10 +42,11 @@ main:
4242
// CHECK-NEXT: Type: SHT_RELA
4343
// CHECK-NEXT: Flags [
4444
// CHECK-NEXT: SHF_ALLOC
45+
// CHECK-NEXT: SHF_INFO_LINK
4546
// CHECK-NEXT: ]
4647
// CHECK-NEXT: Address:
4748
// CHECK-NEXT: Offset:
4849
// CHECK-NEXT: Size: 72
4950

5051
// CHECK: 0x0000000000000008 RELASZ 72
51-
// CHECK: 0x0000000000000002 PLTRELSZ 48
52+
// CHECK: 0x0000000000000002 PLTRELSZ 24

lld/test/ELF/aarch64-combined-dynrel.s

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: AArch64
1+
// REQUIRES: aarch64
22
// RUN: llvm-mc --triple=aarch64-linux-gnu -filetype=obj -o %t.o %s
33
// RUN: echo "SECTIONS { \
44
// RUN: .text : { *(.text) } \
@@ -32,6 +32,7 @@ _start:
3232
// CHECK-NEXT: Type: SHT_RELA
3333
// CHECK-NEXT: Flags [
3434
// CHECK-NEXT: SHF_ALLOC
35+
// CHECK-NEXT: SHF_INFO_LINK
3536
// CHECK-NEXT: ]
3637
// CHECK-NEXT: Address:
3738
// CHECK-NEXT: Offset:

0 commit comments

Comments
 (0)