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-staging] Change assembler to clang in android MonoAOT #110812

Open
wants to merge 4 commits into
base: release/9.0-staging
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 18, 2024

Backport of #110393 to release/9.0-staging

Customer Impact

  • Customer reported
  • Found internally

This change will allows us to bump NDK version to newer one. This will increase security.

Regression

  • Yes
  • No

Testing

This changes were tested with both NDK 23 and NDK 27. Both NDK versions resulted in successful android pipeline runs.

Risk

Low/Medium - this changes the assembler and linker used in android build as per Android official recommendation, this can however result in some unexpected behaviour changes which slipped our tests

Copy link
Contributor

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

@jkurdek jkurdek requested a review from jeffschwMSFT December 18, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants