The existing ELF symbol and section table testing doesn't test many of the corner-cases or valid values for various ELF properties, including things like binding, visibility, section type and so on. This patch adds a series of tests that test these and other related edge-cases.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
Test cases look fine to me, but I would add descriptions about what test is about (where missing), like you did for elf-section-types.test
test/tools/llvm-readobj/elf-symbol-shndx.test | ||
---|---|---|
2 | I would add a test description at first. Because when you start the test from explaining the implementation |
- Add more comments.
- Rebase to remove FIXMEs relating to SHT_MIPS_DWARF.
- Rebase to add required fields for verneed sections.
LGTM
test/tools/llvm-readobj/elf-section-types.test | ||
---|---|---|
1 | nit: all regular maybe? (since machine-specific are tested in elf-machine-specific-section-types.test) |
SHT_MIPS_DWARF should be recognized now after rL354563. Hope nobody complains :)