[all-commits] [llvm/llvm-project] 1a2d2b: [llvm-readobj] - Merge `gnu-symbols.test` to `symb...

Georgii Rymar via All-commits all-commits at lists.llvm.org
Wed Dec 25 04:31:29 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 1a2d2b492a3d16cd9d743c4eb2cafaf11b50092a
      https://github.com/llvm/llvm-project/commit/1a2d2b492a3d16cd9d743c4eb2cafaf11b50092a
  Author: Georgii Rymar <grimar at accesssoftek.com>
  Date:   2019-12-25 (Wed, 25 Dec 2019)

  Changed paths:
    R llvm/test/tools/llvm-readobj/ELF/Inputs/relocs.obj.elf-x86_64
    R llvm/test/tools/llvm-readobj/ELF/Inputs/symbols-proc-specific.elf-hexagon
    R llvm/test/tools/llvm-readobj/ELF/gnu-symbols.test
    M llvm/test/tools/llvm-readobj/ELF/symbols.test

  Log Message:
  -----------
  [llvm-readobj] - Merge `gnu-symbols.test` to `symbols.test` and cleanup.

This cleans up and merges `gnu-symbols.test` to `symbols.test`.
Initially `gnu-symbols.test` tested the following things:
1) How symbols are printed in GNU style.
   It does not make sense to have a separate file for such tests.
2) It tried to test proc-specific symbol indexes. The test was incomplete and
   also we already have `symbol-shndx.test` for that, so this part was removed.
3) It tested `--dyn-symbols` and `--symbols` correlation. All following
   cases were moved to `symbols.test`:
   a) That `--dyn-symbols` does not trigger showing regular symbols..
   b) That `--symbols` triggers `--dyn-symbols` implicitly.
   c) That `--dyn-symbols` and `--symbols` works fine together.

Differential revision: https://reviews.llvm.org/D71697




More information about the All-commits mailing list