[PATCH] D66134: [llvm-size][test] Improve llvm-size testing

[James Henderson via Phabricator via llvm-commits]

jhenderson created this revision.
jhenderson added reviewers: rupprecht, grimar, MaskRay, khemant, davide.
Herald added a project: LLVM.

This patch significantly improves the llvm-size testing. Changes I have made are:

1. Change all ELF tests to use yaml2obj instead of assembly.
2. Move the tests out of the X86 directory, since they don't need to be there after 1).
3. Increased test coverage.
4. Added comments to explain purpose of tests.

I haven't attempted to add test coverage for all Mach-O related code, as I am not familiar enough with that file format to be able to.


Repository:
  rL LLVM

https://reviews.llvm.org/D66134

Files:
  test/tools/llvm-size/X86/elf-sizes.test
  test/tools/llvm-size/X86/format-berkeley-tabs.s
  test/tools/llvm-size/X86/ignore-sections.s
  test/tools/llvm-size/X86/lit.local.cfg
  test/tools/llvm-size/X86/test-common.s
  test/tools/llvm-size/archive.test
  test/tools/llvm-size/basic.test
  test/tools/llvm-size/common.test
  test/tools/llvm-size/darwin-m.test
  test/tools/llvm-size/elf-berkeley.test
  test/tools/llvm-size/elf-m.test
  test/tools/llvm-size/elf-sysv.test
  test/tools/llvm-size/help.test
  test/tools/llvm-size/invalid-input.test
  test/tools/llvm-size/long-format.test
  test/tools/llvm-size/macho-berkeley.test
  test/tools/llvm-size/macho-sysv.test
  test/tools/llvm-size/multiple-inputs.test
  test/tools/llvm-size/no-input.test
  test/tools/llvm-size/radix.test
  test/tools/llvm-size/response-file.test
  test/tools/llvm-size/stdin.test
  test/tools/llvm-size/totals.test
  test/tools/llvm-size/unknown-format.test
  test/tools/llvm-size/version.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66134.214799.patch
Type: text/x-patch
Size: 40664 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190813/1e0c59fd/attachment-0001.bin>