[PATCH] D65807: [obj2yaml] - Move and improve `test\Object\yaml2obj-elf-file-headers-with-e_flags.yaml` test.
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 6 07:46:51 PDT 2019
grimar added a comment.
In D65807#1616916 <https://reviews.llvm.org/D65807#1616916>, @atanasyan wrote:
> I like the idea. But please note that I deleted `yaml2obj-elf-file-headers-with-e_flags.yaml` at rL368023 <https://reviews.llvm.org/rL368023> because the same functionality is checked by the `Mips/elf-flags.yaml`. When this patch is applied or as a part of the applying, we can delete `Mips/elf-flags.yaml` as well.
Ok, cool. I did not notice them this time. I see there are 2 tests related there: `Mips/elf-abi.yaml` and `Mips/elf-abi.yaml`.
Both of them are incomplete (do not cover all the possible values) and seems contain too much excessive things.
(like section `.text` or symbol). I am going to remove these 2 in the updated diff.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65807/new/
https://reviews.llvm.org/D65807
More information about the llvm-commits
mailing list