Skip to content

[flang][runtime] Fix RU/RD results when rounding to least nonzero #75878

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
Dec 26, 2023

Conversation

klausler
Copy link
Contributor

When rounding what otherwise would have been a result that underflowed to zero up (RU) or down (RD) to the least magnitude nonzero subnormal number, ensure that the original exponent value doesn't perturb the result.

@klausler klausler requested a review from psteinfeld December 19, 2023 00:28
@llvmbot llvmbot added the flang Flang issues not falling into any other category label Dec 19, 2023
@klausler klausler requested review from vdonaldson and removed request for psteinfeld December 19, 2023 00:28
Copy link
Contributor

@psteinfeld psteinfeld left a comment

Choose a reason for hiding this comment

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

All builds and tests correctly.

@klausler klausler force-pushed the bug1455 branch 2 times, most recently from 145640f to 782c255 Compare December 26, 2023 23:52
Copy link

github-actions bot commented Dec 26, 2023

✅ With the latest revision this PR passed the C/C++ code formatter.

When rounding what otherwise would have been a result that underflowed
to zero up (RU) or down (RD) to the least magnitude nonzero subnormal
number, ensure that the original exponent value doesn't perturb the
result.
@klausler klausler merged commit f45723c into llvm:main Dec 26, 2023
@klausler klausler deleted the bug1455 branch December 26, 2023 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flang Flang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants