Mac Catalyst has different platform fields in build_version_command,
so it's natural to create a separate libclang_rt.
Details
- Reviewers
yln delcypher smeenai aralisza kubamracek
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Time | Test | |
---|---|---|
290 ms | x64 debian > Clang.Driver::darwin-sanitizer-ld.c |
Event Timeline
@bc-lee Thanks for the patch. While I get what you're trying to do I have doubts about being able to accept the patch in its current form. Apple's ASan catalyst doesn't work by building a separate dylib, instead it builds the macosx dylib in a different way to make it work with catalyst code.
I've made @aralisza a blocking reviewer because she'll need to approve this.
It may not be appropriate to add other runtime libraries specifically for Mac Catalyst. However, currently lld does not allow linking with dynamic libraries with different types of build_version_command, whereas ld64 does allow for Mac Catalyst. Considering compatibility with lld, Mac Catalyst’s runtime libraries need to be added also.
See my other patch https://reviews.llvm.org/D117925 for more details.
If lld doesn't support something we need, we should add support for the missing part to lld instead of working around it. Is there a bug on file for the missing lld bit?
For the Chromium side issue is https://crbug.com/1259122. Also I opened LLVM issue https://github.com/llvm/llvm-project/issues/53477.
Some changes in this patch were applied as https://reviews.llvm.org/D118759, so I'm re-uploading this patch to get rid of merge conflicts.
Closed in favor of https://reviews.llvm.org/D118862 and https://reviews.llvm.org/D118875