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

[Bug Report] Incorrect upward rounding result of fsub.s instruction #84

Open
zhangkanqi opened this issue Mar 18, 2024 · 3 comments
Open

Comments

@zhangkanqi
Copy link

Hi,

When executing the fsub.s ft5, ft5, ft2, rup instruction in Spike, I found that the result is incorrect, where ft5 is initialized as 0x284d52df and ft2 is initialized as 0x7b418a35.
I believe the correct final result in ft5 should be 0xfb418a34 (as Spike does), however, the result calculated in RSD is 0xfb418a35.

@lpha-z
Copy link

lpha-z commented Mar 18, 2024

Thanks for your bug report.
The current implementation of the floating-point arithmetic unit does not support directional rounding (so really it does not fully support RV32F).
We plan to add support for directional rounding in the near future.

@zhangkanqi
Copy link
Author

I see. Thanks a lot.

@lpha-z
Copy link

lpha-z commented Mar 18, 2024

We reopen this report because we believe it will help others who have faced similar problems.
When we finish supporting the directional rounding, we will close this report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants