Skip to content

Commit

Permalink
[LLD][ELF]Fix tests for D56910
Browse files Browse the repository at this point in the history
r351789 changes the output of llvm-readelf --dyn-symbols. This causes 3
LLD tests to break. This patch fixes them.

Reviewed by: ruiu

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

llvm-svn: 351790
  • Loading branch information
jh7370 committed Jan 22, 2019
1 parent 5fc812f commit b88d6fd
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
7 changes: 4 additions & 3 deletions lld/test/ELF/as-needed-weak.s
Expand Up @@ -10,9 +10,10 @@

# CHECK-NOT: libfoo

# CHECK: Symbol table of .hash for image:
# CHECK-NEXT: Num Buc: Value Size Type Bind Vis Ndx Name
# CHECK-NEXT: 1 1: 0000000000000000 0 FUNC WEAK DEFAULT UND foo
# CHECK: Symbol table '.dynsym' contains 2 entries:
# CHECK-NEXT: Num: Value Size Type Bind Vis Ndx Name
# CHECK-NEXT: 0: 0000000000000000 0 NOTYPE LOCAL DEFAULT UND
# CHECK-NEXT: 1: 0000000000000000 0 FUNC WEAK DEFAULT UND foo

.globl _start
.weak foo
Expand Down
2 changes: 1 addition & 1 deletion lld/test/ELF/icf-symbol-type.s
Expand Up @@ -7,8 +7,8 @@
# We used to mark bar as absolute.

# CHECK: .text PROGBITS 0000000000001000
# CHECK: 0000000000001001 0 NOTYPE GLOBAL DEFAULT 4 foo
# CHECK: 0000000000001001 0 NOTYPE GLOBAL DEFAULT 4 bar
# CHECK: 0000000000001001 0 NOTYPE GLOBAL DEFAULT 4 foo

# The nop makes the test more interesting by making the offset of
# text.f non zero.
Expand Down
2 changes: 1 addition & 1 deletion lld/test/ELF/linkerscript/provide-shared2.s
Expand Up @@ -6,7 +6,7 @@
# RUN: ld.lld -o %t --script %t.script %t.o %t2.so
# RUN: llvm-readelf --dyn-symbols %t | FileCheck %s

# CHECK: 1 1: 000000000000002a 0 NOTYPE GLOBAL DEFAULT ABS foo
# CHECK: 1: 000000000000002a 0 NOTYPE GLOBAL DEFAULT ABS foo

.global _start
_start:
Expand Down

0 comments on commit b88d6fd

Please sign in to comment.