This is an archive of the discontinued LLVM Phabricator instance.

[LLD][ELF] - Convert invalid/undefined-local-symbol-in-dso.test to use YAML
ClosedPublic

Authored by grimar on Feb 28 2019, 6:25 AM.

Details

Summary

This is a big well-commented test, so I am posting it in a separate review
from D58780.

But the idea of the patch is the same:
this depends on D58779 and allows us to remove a binary from inputs.

Diff Detail

Repository
rL LLVM

Event Timeline

grimar created this revision.Feb 28 2019, 6:25 AM
grimar updated this revision to Diff 188735.Feb 28 2019, 6:28 AM
  • Restored a part of original comment with the description of the issue.
ruiu accepted this revision.Feb 28 2019, 11:06 AM

LGTM

Thanks!

This revision is now accepted and ready to land.Feb 28 2019, 11:06 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2019, 2:39 AM