Skip to content

[release/8.0] Fix compilation of runtime with Xcode 16 - #112208

Merged
jkotas merged 3 commits into
dotnet:release/8.0-stagingfrom
filipnavara:backport/8.0/xcode-16
Feb 5, 2025
Merged

[release/8.0] Fix compilation of runtime with Xcode 16#112208
jkotas merged 3 commits into
dotnet:release/8.0-stagingfrom
filipnavara:backport/8.0/xcode-16

Conversation

@filipnavara

Copy link
Copy Markdown
Member

Backport of #106744, #106442, and #111530 to release/8.0

Ref: dotnet/sdk#46006 (comment)

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 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.

Regression

  • Yes
  • No

Testing

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

Risk

Low

@ghostghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Feb 5, 2025
@dotnet-policy-servicedotnet-policy-serviceBot added the community-contribution Indicates that the PR has been added by a community member label Feb 5, 2025
@jkotasjkotas added area-NativeAOT-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Feb 5, 2025
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

@jkotasjkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

approved. we can treat this as tell mode. we can merge when ready

cc @carlossanlop

@jeffschwMSFTjeffschwMSFT added the Servicing-approved Approved for servicing release label Feb 5, 2025
@jeffschwMSFTjeffschwMSFT added this to the 8.0.14 milestone Feb 5, 2025
@jkotas
jkotas merged commit 41e66d3 into dotnet:release/8.0-stagingFeb 5, 2025
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Mar 8, 2025
@filipnavara
filipnavara deleted the backport/8.0/xcode-16 branch April 2, 2025 19:55
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-NativeAOT-coreclrcommunity-contributionIndicates that the PR has been added by a community memberServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@filipnavara@jkotas@jeffschwMSFT