This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][nfc] Rename some tests for consistency
ClosedPublic

Authored by int3 on Mar 3 2022, 2:04 PM.

Details

Reviewers
thakis
Group Reviewers
Restricted Project
Commits
rG984197612ce9: [lld-macho][nfc] Rename some tests for consistency
Summary

Now all the tests that cover symbol resolution / precedence have
"resolution" in their filename.

I also added a couple of extra comments.

Diff Detail

Event Timeline

int3 created this revision.Mar 3 2022, 2:04 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 3 2022, 2:04 PM
int3 requested review of this revision.Mar 3 2022, 2:04 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 3 2022, 2:04 PM
thakis accepted this revision.Mar 3 2022, 6:19 PM
thakis added a subscriber: thakis.

Meh, i find "order" much more descriptive than "resolution" for the order tests.

This revision is now accepted and ready to land.Mar 3 2022, 6:19 PM
int3 added a comment.Mar 3 2022, 6:25 PM

"order" makes me think of "order files"...

This revision was automatically updated to reflect the committed changes.