[PATCH] D64555: [test/Object, obj2yaml] - Move test cases from test/Object and cleanup.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 04:03:16 PDT 2019


grimar created this revision.
grimar added reviewers: jhenderson, MaskRay.

`test/Object` is not correct place to have tests that check `obj2yaml`
functionality, because we have `test/tools/obj2yaml` folder for that.

In this patch I merged a few test cases with their YAMLs from Inputs
folder, converted one of binary inputs and moved them to
`tools/obj2yaml` folder.

There are other tests that might need to do the same, so it is initial step. 
My motivation was that I was able to damage a code responding for dumping/reading
group sections and no tests in yaml2obj/ob2yaml failed, what was really
confusing to see.


https://reviews.llvm.org/D64555

Files:
  test/Object/Inputs/COFF/long-file-symbol.yaml
  test/Object/Inputs/COFF/long-section-name.yaml
  test/Object/Inputs/COFF/section-aux-symbol.yaml
  test/Object/Inputs/COFF/weak-external.yaml
  test/Object/Inputs/invalid-reloc.elf-x86-64
  test/Object/obj2yaml-coff-long-file-symbol.test
  test/Object/obj2yaml-coff-long-section-name.test
  test/Object/obj2yaml-coff-section-aux-symbol.test
  test/Object/obj2yaml-coff-weak-external.test
  test/Object/obj2yaml-invalid-reloc.test
  test/Object/obj2yaml-sectiongroup.test
  test/tools/obj2yaml/coff-long-file-symbol.test
  test/tools/obj2yaml/coff-long-section-name.test
  test/tools/obj2yaml/coff-section-aux-symbol.test
  test/tools/obj2yaml/coff-weak-external.test
  test/tools/obj2yaml/invalid-reloc.test
  test/tools/obj2yaml/section-group.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64555.209161.patch
Type: text/x-patch
Size: 18079 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190711/74cd1286/attachment.bin>


More information about the llvm-commits mailing list