This patch adds a number of tests to test various cases not covered by existing tests. All of them work correctly, with no need to change llvm-objcopy itself, although some do indicate possible areas for improvement.
Details
Details
- Reviewers
jakehehrlich evgeny777 rupprecht grimar MaskRay • espindola alexander-shaposhnikov - Commits
- rZORGe028058b8123: [llvm-objcopy] Add additional testing for various cases
rZORG758c32640f0a: [llvm-objcopy] Add additional testing for various cases
rGe028058b8123: [llvm-objcopy] Add additional testing for various cases
rG758c32640f0a: [llvm-objcopy] Add additional testing for various cases
rG5772e02bd0f1: [llvm-objcopy] Add additional testing for various cases
rL360422: [llvm-objcopy] Add additional testing for various cases
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Address review comments:
- Add extra spacing.
- Rename tail merging test/update comments.
- Add extra symbol and section to tail merging test.
- Remove unnecessary switches from tail merging test.
llvm/trunk/test/tools/llvm-objcopy/ELF/shared-strtab-shstrtab.s | ||
---|---|---|
63 ↗ | (On Diff #199006) | This makes the test fail if "foo" is a substring of the path of your build dir. Could you make this more robust against accidentally matching the directory name? |
'r+b' if you want to make it work on Python 3 as well though.