This is an archive of the discontinued LLVM Phabricator instance.

Wrap xar/xar.h include in extern "C" block
ClosedPublic

Authored by glandium on Sep 22 2021, 12:57 AM.

Details

Reviewers
gkm
thakis
Group Reviewers
Restricted Project
Commits
rG08ef24f6abdf: Wrap xar/xar.h include in extern "C" block
Summary

Without such wrapping, linking lld fails with missing symbols because of
C++ symbol mangling with older versions of the MacOSX SDK, in which
xar.h doesn't have an extern "C" block itself.

Diff Detail

Event Timeline

glandium created this revision.Sep 22 2021, 12:57 AM
Herald added a project: Restricted Project. · View Herald Transcript
glandium requested review of this revision.Sep 22 2021, 12:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2021, 12:57 AM
glandium edited the summary of this revision. (Show Details)Sep 22 2021, 3:47 AM
thakis accepted this revision.Sep 22 2021, 10:09 AM
thakis added a subscriber: thakis.

lg.

out of interest, which SDK version has this problem?

This revision is now accepted and ready to land.Sep 22 2021, 10:09 AM

lg.

out of interest, which SDK version has this problem?

10.12 is apparently the last one with this problem. (yeah, I know, that's old)

This revision was automatically updated to reflect the committed changes.