Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable/20240723][Darwin][Driver][clang] arm64-apple-none-macho is missing the Apple macros from arm-apple-none-macho #9819

Conversation

ian-twilightcoder
Copy link

arm-apple-none-macho uses DarwinTargetInfo which provides several Apple specific macros. arm64-apple-none-macho however just uses the generic AArch64leTargetInfo and doesn't get any of those macros. It's not clear if everything from DarwinTargetInfo is desirable for arm64-apple-none-macho, so make an AppleMachOTargetInfo to hold the generic Apple macros and a few other basic things.

rdar://142625255

…ssing the Apple macros from arm-apple-none-macho

arm-apple-none-macho uses DarwinTargetInfo which provides several Apple specific macros. arm64-apple-none-macho however just uses the generic AArch64leTargetInfo and doesn't get any of those macros. It's not clear if everything from DarwinTargetInfo is desirable for arm64-apple-none-macho, so make an AppleMachOTargetInfo to hold the generic Apple macros and a few other basic things.

rdar://142625255
@ian-twilightcoder ian-twilightcoder requested a review from a team as a code owner January 11, 2025 00:08
@ian-twilightcoder
Copy link
Author

@swift-ci test llvm

@ian-twilightcoder
Copy link
Author

@swift-ci test

@ian-twilightcoder
Copy link
Author

llvm Transforms/Util/auto-init-annotation-remarks.ll failure is unrelated.

@ian-twilightcoder ian-twilightcoder merged commit e67c0c3 into swiftlang:stable/20240723 Jan 11, 2025
3 of 5 checks passed
@ian-twilightcoder ian-twilightcoder deleted the apple-target-info branch January 15, 2025 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant