[llvm-readobj][test] - Cleanup testing of the --sections command line option.
authorGeorgii Rymar <grimar@accesssoftek.com>
Wed, 29 Jan 2020 14:10:07 +0000 (17:10 +0300)
committerGeorgii Rymar <grimar@accesssoftek.com>
Fri, 31 Jan 2020 09:58:12 +0000 (12:58 +0300)
commitcf6037b5619037be993ee9d5850737f25cdd82c1
treec3e54a9bbe02dff25e2a7a57cc08d54e066216aa
parent3f6a2f1ec520651305beaef79fc3f19ebf4ebda6
[llvm-readobj][test] - Cleanup testing of the --sections command line option.

We have the `ELF\sections.test` to test --sections.

`ELF\sections.test` uses precompiled objects, it has a bug (does not test -s alias properly).
Also, we test machine specific section types in `ELF\machine-specific-section-types.test`,
so we probably do not need to test `--sections` for a MIPS object in `ELF\sections.test`.
I think it is enough to test ELF32 and ELF64 (we do not test ELF64 in this test).

`Object/readobj-shared-object.test` also tests how llvm-readobj handles
`--sections`. It is location is wrong, it is not complete, it uses precompiled binaries
and it duplicates the `ELF\sections.test` partially (it tests both ELF32 and ELF64).

We have `ELF\readelf-s-alias.test` that tests the `-s` alias for `--sections` in llvm-readobj
and `-s` as an alias for `--symbols` in llvm-readelf.
There is no need to have a separate test for such things.
The test for the `-s` alias for `--sections` can be included into the `ELF\sections.test`.
And the test for `-s` for llvm-readelf is already included into `ELF\symbols.test`.

So, this patch:
1) Removes `Object/readobj-shared-object.test`.
2) Removes `ELF\readelf-s-alias.test`
3) Rewrites the `ELF\sections.test`.
4) Removes ELF/Inputs/trivial.obj.elf-mipsel.

Differential revision: https://reviews.llvm.org/D73686
llvm/test/Object/readobj-shared-object.test [deleted file]
llvm/test/tools/llvm-readobj/ELF/Inputs/trivial.obj.elf-mipsel [deleted file]
llvm/test/tools/llvm-readobj/ELF/readelf-s-alias.test [deleted file]
llvm/test/tools/llvm-readobj/ELF/sections.test