[PATCH] D57518: [llvm-symbolizer][test] Rename and tweak tests using llvm-symbolizer

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 09:11:51 PST 2019


jhenderson created this revision.
jhenderson added a reviewer: dblaikie.

There are a few tests called llvm-symbolizer* in the DebugInfo test area. These really are testing either the DebugInfo or Symbolizer library, rather than the llvm-symbolizer tool itself, so this patch renames them to be clearer that they aren't explicitly tests for llvm-symbolizer (such tests belong in test/tools/llvm-symbolizer).

This patch also reinstates the copying of a DWO file, removed previously in rL352752 <https://reviews.llvm.org/rL352752>. The test needs this so that it could possibly fail.

Finally, some of the tests have been simplified slightly by removing unnecessary switches and/or unused check-prefixes.


Repository:
  rL LLVM

https://reviews.llvm.org/D57518

Files:
  test/DebugInfo/llvm-symbolizer-split-dwarf-empty.test
  test/DebugInfo/llvm-symbolizer-split-dwarf-no-skel-address.test
  test/DebugInfo/llvm-symbolizer-tu.test
  test/DebugInfo/llvm-symbolizer-zlib.test
  test/DebugInfo/llvm-symbolizer.test
  test/DebugInfo/symbolize-split-dwarf-empty.test
  test/DebugInfo/symbolize-split-dwarf-no-skel-address.test
  test/DebugInfo/symbolize-tu.test
  test/DebugInfo/symbolize-zlib.test
  test/DebugInfo/symbolize.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57518.184520.patch
Type: text/x-patch
Size: 6489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190131/322eb47d/attachment-0001.bin>


More information about the llvm-commits mailing list