diff options
| author | Georgii Rymar <grimar@accesssoftek.com> | 2020-01-29 17:10:07 +0300 |
|---|---|---|
| committer | Georgii Rymar <grimar@accesssoftek.com> | 2020-01-31 12:58:12 +0300 |
| commit | cf6037b5619037be993ee9d5850737f25cdd82c1 (patch) | |
| tree | c3e54a9bbe02dff25e2a7a57cc08d54e066216aa /llvm/lib/IR/Module.cpp | |
| parent | 3f6a2f1ec520651305beaef79fc3f19ebf4ebda6 (diff) | |
| download | llvm-cf6037b5619037be993ee9d5850737f25cdd82c1.zip llvm-cf6037b5619037be993ee9d5850737f25cdd82c1.tar.gz llvm-cf6037b5619037be993ee9d5850737f25cdd82c1.tar.bz2 | |
[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
Diffstat (limited to 'llvm/lib/IR/Module.cpp')
0 files changed, 0 insertions, 0 deletions