

fcvt.bf16.s encoding collides with fround.h from zfa · Issue #33 · riscv/riscv-b...
source link: https://github.com/riscv/riscv-bfloat16/issues/33
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.

fcvt.bf16.s encoding collides with fround.h from zfa #33
Closed
asb opened this issue Mar 27, 2023 · 12 comments
Comments
Contributor
The Zfa extension describes
|
Contributor
Author
Just a ping on this. |
Collaborator
Thanks for pointing this out Alex. I'm looking into a new encoding for fcvt.bf16.s. |
Collaborator
fcvt.bf16.s is now encoded like FCVT.H.S, but with rs2=8 |
@kdockser Could I ask you to roll a new release artifact with this change and a new version number? Having a cite-able PDF with a distinct version will make it easier to do change tracking for downstream projects. In particular, this should unblock us landing experimental support in LLVM. |
Collaborator
Reopened until the change is pushed (which will be soon). |
Contributor
Author
I see the change is pushed now (thanks!), but a new PDF build and an incremented spec version number would be much appreciated. I could commit the experimental support for bfloat16 with reference to the current commit id, but it's not ideal that people wanting to review the implementation can't easily access a PDF. |
Contributor
Author
The spec number is now incremented (thanks!) - would it be possible to get a new PDF release? |
Collaborator
A new PDF has been released. This includes that changes suggested by the AR Committee. |
Contributor
Author
Contributor
Author
Another ping, as the PDF still doesn't seem to be posted. Many thanks. |
Contributor
Author
It seems a new PDF was posted last week, so this request can be considered answered. Thank you. |
Contributor
Author
With the PDF now available, I've been able to update the LLVM assembler/disassembler patches based on the v0.6 version of the spec:
|
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
No one assigned
None yet
No milestone
No branches or pull requests
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK