We have the elf-dynamic-tags-machine-specific.yaml input shared
between the llvm-readobj and llvm-objdump test.
It looks strange, because tools usually does not share inputs.
Also there are following problems related:
- elf-dynamic-tags-machine-specific.yaml input contains excessive YAML parts.
- objdump's test case never test AARCH64 tags.
- There are unknown tags in the elf-dynamic-tags-machine-specific.yaml and dynamic-tags-machine-specific.test, though we already testing unknown tags in llvm-readobj\ELF\dynamic-tags.test and llvm-objdump\elf-dynamic-section.test tests.
This patch removes the shared input and refines the test cases to resolve
issues mentioned.
I'd suggest some wording changes:
And I wouldn't put a blank line after the comments at the start of each machine case, because the comment is tied to the respective RUN lines (just like you wouldn't put a comment for an if statement in C++ with a blank line between it and the if).