This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][test] Add -lSystem to all lld invocations
ClosedPublic

Authored by keith on Oct 4 2022, 1:48 PM.

Details

Reviewers
int3
Group Reviewers
Restricted Project
Commits
rG0634a8440d63: [lld-macho][test] Add -lSystem to all lld invocations
Summary

This is a requirement from ld64, so this mirrors that expectation. Most
of these changes were just offsets from adding the library, or adding
dyld_stub_binder which is always added when linking ld64. This also adds
%no-lsystem-lld for the few cases where we're explicitly testing
things without that.

Fixes: https://github.com/llvm/llvm-project/issues/54184

Diff Detail

Event Timeline

keith created this revision.Oct 4 2022, 1:48 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 4 2022, 1:48 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
keith requested review of this revision.Oct 4 2022, 1:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2022, 1:48 PM
int3 accepted this revision.Oct 4 2022, 2:07 PM
int3 added a subscriber: int3.

lgtm, thanks!

This revision is now accepted and ready to land.Oct 4 2022, 2:07 PM
keith added inline comments.Oct 4 2022, 2:12 PM
lld/test/MachO/arm-dtrace.s
4

This case is only a problem since our fake libSystem doesn't support these architectures, especially armv4t. I can add them if that's the right move

This revision was landed with ongoing or failed builds.Oct 4 2022, 2:21 PM
This revision was automatically updated to reflect the committed changes.