This is an archive of the discontinued LLVM Phabricator instance.

[lld][MachO][NFC] Pull out code into helper function.
AbandonedPublic

Authored by Roger on Nov 29 2021, 12:29 PM.

Details

Reviewers
int3
Group Reviewers
Restricted Project
Summary

This is currently in a separate commit for the ease of review.
In the final commits, this will likely be merged into one diff that
includes the functional changes.

Diff Detail

Event Timeline

Roger created this revision.Nov 29 2021, 12:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 29 2021, 12:29 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Roger updated this revision to Diff 394955.Dec 16 2021, 12:14 PM

small fix.

Roger updated this revision to Diff 395004.Dec 16 2021, 2:50 PM

Compilation fix.

Roger updated this revision to Diff 396049.Dec 23 2021, 10:29 AM

test passes

Roger published this revision for review.Dec 23 2021, 10:34 AM
Roger edited the summary of this revision. (Show Details)
Herald added a project: Restricted Project. · View Herald TranscriptDec 23 2021, 10:35 AM
Roger added a reviewer: int3.Dec 23 2021, 10:35 AM
int3 added a comment.Dec 25 2021, 2:26 AM

eh I don't think it's necessary to make your changes this granular even for ease of review. this can def be folded into a larger change

int3 requested changes to this revision.Jan 5 2022, 1:46 AM

clearing my queue

This revision now requires changes to proceed.Jan 5 2022, 1:46 AM
Roger abandoned this revision.Jan 6 2022, 11:46 AM