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

[release/9.0] Fix compilation of runtime with Xcode 16 #106800

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

filipnavara
Copy link
Member

Backport of #106744 and #106442 to release/9.0

/cc @jkotas

Customer Impact

  • Customer reported
  • Found internally

Apple build tools (Xcode) are updated annually. Customers are expected to use up to date version to build applications, which is often enforced through policies on Apple systems and tooling (eg. uploading to App Store or signing and notarizing may enforce minimum Xcode version necessary for building apps). Xcode 16, currently at beta 6, started enforcing restriction on correct layout of unwinding information in relation to publicly exported symbols. In order to allow compilation with the new tooling we need to mark some of the runtime symbols with .alt_entry to signify they are not function entrypoints but rather just labels pointing at specific instruction in a given function. Apple has a release scheduled closely aligned with the .NET release schedule with the major releases happening in autumn. We should ensure the .NET 9 release compiles with the up-to-date tooling at the time of the release.

Regression

  • Yes
  • No

Testing

The runtime repository was built on Apple machine with Xcode 16 beta 6 and it no longer produces compilation errors. NativeAOT smoke tests were used to verify that the code still links and runs as it should.

Risk

Low

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Aug 22, 2024
Copy link
Contributor

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved. we can merge when ready

@jeffschwMSFT jeffschwMSFT added the Servicing-approved Approved for servicing release label Aug 22, 2024
@jkotas jkotas merged commit 562efd6 into dotnet:release/9.0 Aug 22, 2024
88 of 95 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 22, 2024
@filipnavara filipnavara deleted the backport-alt-entry branch October 13, 2024 19:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-NativeAOT-coreclr community-contribution Indicates that the PR has been added by a community member Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants