This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][nfc] Use split-file in order file test
ClosedPublic

Authored by int3 on Dec 18 2020, 3:43 PM.

Details

Reviewers
smeenai
Group Reviewers
Restricted Project
Commits
rGe2863357de77: [lld-macho][nfc] Use split-file in order file test
Summary

Makes it much easier to edit / maintain.

Diff Detail

Event Timeline

int3 requested review of this revision.Dec 18 2020, 3:43 PM
int3 created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptDec 18 2020, 3:43 PM
smeenai accepted this revision.Dec 18 2020, 3:56 PM
smeenai added a subscriber: smeenai.

It's a little confusing to have the RUN lines inside the splits IMO. It works fine cos they'll get treated as comments, but I think I'd find this easier to read if you had all the RUNs followed by the order file splits instead of them being interspersed.

This revision is now accepted and ready to land.Dec 18 2020, 3:56 PM
int3 updated this revision to Diff 312988.Dec 20 2020, 10:47 AM

reorganize order files at bottom of test

This revision was landed with ongoing or failed builds.Dec 20 2020, 10:49 AM
This revision was automatically updated to reflect the committed changes.