This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by jhenderson on Jan 30 2019, 4:52 AM.

Details

Summary

This is the third of a series of patches relating to simplifying llvm-symbolizer tests. See D57441 and D57443 for the previous two. In this one, I have split out a number of distinct test cases from the llvm-symbolizer.test test into separate tests, and simplified them in various ways including:

  1. by using --obj/positional arguments for the input file and addresses instead of stdin,
  2. using runtime-generated inputs rather than a pre-canned binary, and
  3. testing more specifically (i.e. checking only what is interesting to the behaviour changed in the original commit for that test case).

I've removed the test case for using --obj. The tools/llvm-symbolizer/basic.s already tests this case. I've also added a simple test case to the demangle switch test case to show that demangling happens by default.

See PR40070 comment 1 onwards for the motivation.

Diff Detail

Repository
rL LLVM

Event Timeline

jhenderson created this revision.Jan 30 2019, 4:52 AM
dblaikie accepted this revision.Jan 30 2019, 10:35 AM

Thanks!

This revision is now accepted and ready to land.Jan 30 2019, 10:35 AM
This revision was automatically updated to reflect the committed changes.

I accidentally committed r352756 referring to this review instead of D57451.