This is an archive of the discontinued LLVM Phabricator instance.

[llvm-symbolizer][test] Extract tests from llvm-symbolizer.test and simplify (#3)
ClosedPublic

Authored by jhenderson on Jan 30 2019, 7:28 AM.

Details

Summary

This is the fourth (and final for now) of a series of patches relating to simplifying llvm-symbolizer tests. See D57441, D57443 and D57446 for the previous ones. In this one, I have split out several more distinct test cases from the llvm-symbolizer.test test into separate tests, and simplified them in various ways including:

  1. Building a test case for spaces in path from source, rather than using a pre-canned binary. This allows deleting of said binary and the source it was built from.
  2. Switched to specifying addresses and objects directly on the command-line rather than via stdin.

This also adds an explict test for the ability to specify a file and address as a line in stdin, since the majority of our tests are being migrated away from this approach.

Diff Detail

Repository
rL LLVM

Event Timeline

jhenderson created this revision.Jan 30 2019, 7:28 AM
dblaikie accepted this revision.Jan 30 2019, 10:36 AM

Thanks!

This revision is now accepted and ready to land.Jan 30 2019, 10:36 AM