michaeleisel / zld

A faster version of Apple's linker
MIT License
1.19k stars 50 forks source link

Upgrade TAPI to fix issues with Catalyst #50

Closed michaeleisel closed 4 years ago

michaeleisel commented 4 years ago

This replaces the auto-downloaded version of tapi with a source-controlled version created by downloading tapi-1000.10.8 from Apple open source and following its readme directions. However, Catalyst still isn't supported with this one, so it needs some additional changes in Version.inc.h to support the linker's use of it. My biggest concern is using iOSMac = 6 and zippered = UINT_MAX, as there's no proof that this is what those values are supposed to be. I also had to leave the use of getPlatformSet disabled, as the method isn't used in the latest version of tapi. This is a hacky solution, but I wonder when tapi will ever get an open source release again, considering it's only had two.

michaeleisel commented 4 years ago

i'm also in contact with someone with apple open source, maybe they'll do a release soon

michaeleisel commented 4 years ago

it looks like a release is probably not forthcoming (although a release of xcode 12 tapi's headers after its release seems quite possible). so, i think we'll want to try this version for now. it's a pretty straightforward format in many ways, so i think we can make it work. @steipete did you have a chance to try this patch?

michaeleisel commented 4 years ago

mitigated with https://github.com/michaeleisel/zld/pull/55