Skip to content

Update the ABI to support Floating-Point Exceptions. #5164

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

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

pmsjt
Copy link
Contributor

@pmsjt pmsjt commented Jan 29, 2025

Update the ABI to be in concordance, with the introduction of the first officially supported CPU with FP exceptions.

While the original rule was a good intention, it was at odds with the Arm ABI, which is also adopted by Linux and macOS, making code harder to port.

It was also at odds with the Arm Architecture in the sense that if offers a direct way to convey to software when FP exceptions are supported and when they are enabled, and this rule was negating such contract.

…st officially supported CPU with FP exceptions.

While the original rule was a good intention, it was at odds with the Arm ABI, which is also adopted by Linux and macOS, making code harder to port.
It was also at odds with the Arm Architecture in the sense that if offers a direct way to convey to software when FP exceptions are supported and when they are enabled, and this rule was negating such contract.
Copy link
Contributor

@pmsjt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit ea29a83:

✅ Validation status: passed

File Status Preview URL Details
docs/build/arm64-windows-abi-conventions.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Collaborator

@TylerMSFT TylerMSFT 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, @pmsjt. When you are ready for this to get merged, add #sign-off in a comment.

@v-dirichards
Copy link
Contributor

Thank you, @pmsjt. When you are ready for this to get merged, add #sign-off in a comment.

@TylerMSFT I'll merge this PR based on your approval. If this article requires further changes, please open a separate pull request.

@v-dirichards v-dirichards merged commit ce24833 into MicrosoftDocs:main Jan 29, 2025
2 checks passed
@pmsjt pmsjt deleted the patch-2 branch February 4, 2025 08:21
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