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

Sync with spike version of softfloat #26

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

troibe
Copy link

@troibe troibe commented Feb 24, 2024

Addresses #24
It adds the following modifications of softfloat version 3d from Spike:

  • Additional support for fp16 and bf16 conversions
  • Support for fmax/fmin, reciprocal, rsqrt

As far as I can tell all of these were added for the RISC-V vector extension.
Any modifications related to the build system in Spike were left out.

As I'm just a user and by no means a softfloat expert, I would appreciate some advice on these things before merging:

  • I'm assuming the changes in specialization.h were specific to RISC-V. What would be the quickest way of optaining the special values for ARM and 8086? Then I'll adjust them from the RISC-V vaues which I currently just copied.
  • Were the unused paramater/variable warnings specific to the build system used in Spike (I couldn't reproduce them here)?

@troibe troibe force-pushed the sync-with-spike-version-3d branch 2 times, most recently from 390fa46 to 8c8f25e Compare February 28, 2024 23:20
@troibe troibe force-pushed the sync-with-spike-version-3d branch from 8c8f25e to 6ffd8a8 Compare February 28, 2024 23:26
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

Successfully merging this pull request may close these issues.

4 participants