Skip to content

Commit 5f6ff71

Browse files
author
Georgii Rymar
committed
[llvm-readobj][test] - Add a check for --dyn-syms.
We have a `-dyn-symbols` option. It has a `--dyn-syms` alias that is not tested in llvm-readobj tests currently. There was a Object/readobj-elf-versioning.test where it is used, but I've removed it in D73163. And also it is not the right place to test it anyways. Differential revision: https://reviews.llvm.org/D73164
1 parent 46acce4 commit 5f6ff71

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

llvm/test/tools/llvm-readobj/ELF/dyn-symbols.test

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,19 @@ ProgramHeaders:
6969
Sections:
7070
- Section: .dynsym
7171

72-
## Case 2: Check the two-letter alias --dt is equivalent to the --dyn-symbols
72+
## Case 2: Check the output for aliases.
73+
## a) Check the two-letter alias --dt is equivalent to the --dyn-symbols
7374
## full flag name.
7475
# RUN: llvm-readobj --dt %t1.so > %t.readobj-dt-alias
7576
# RUN: llvm-readobj --dyn-symbols %t1.so > %t.readobj-dt-no-alias
7677
# RUN: cmp %t.readobj-dt-alias %t.readobj-dt-no-alias
78+
## b) Check --dyn-syms equals --dyn-symbols, --dt for llvm-readobj.
79+
# RUN: llvm-readobj --dyn-syms %t1.so > %t.readobj-dyn-syms
80+
# RUN: cmp %t.readobj-dt-alias %t.readobj-dyn-syms
81+
## c) Check --dyn-syms equals --dyn-symbols for llvm-readelf.
82+
# RUN: llvm-readelf --dyn-syms %t1.so > %t.readelf-dyn-syms
83+
# RUN: llvm-readelf --dyn-symbols %t1.so > %t.readelf-dyn-symbols
84+
# RUN: cmp %t.readelf-dyn-symbols %t.readelf-dyn-syms
7785

7886
## Case 3: Check that we are able to dump the dynamic symbol table even when we have no program headers.
7987
## In this case we find the table by it's type (SHT_DYNSYM) and ignore the DT_SYMTAB value.

0 commit comments

Comments
 (0)