This is an archive of the discontinued LLVM Phabricator instance.

[llvm-ar][test] Add to llvm-ar test coverage
ClosedPublic

Authored by gbreynoo on Jan 24 2020, 12:58 AM.

Details

Summary

This diff is depends on D69665.

  • Added handling of thin archives to symtab.test.
  • Added handling of newlines to response.test.
  • D69665 exposed behaviour regarding the use of -- on the command line. Added double-hyphen.test to cover this.

Diff Detail

Event Timeline

gbreynoo created this revision.Jan 24 2020, 12:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 24 2020, 12:58 AM
MaskRay accepted this revision.Jan 24 2020, 10:34 AM
MaskRay added inline comments.
llvm/test/tools/llvm-ar/symtab.test
58

Join the two lines. It will not be too long.

This revision is now accepted and ready to land.Jan 24 2020, 10:34 AM
This revision was automatically updated to reflect the committed changes.