-
Notifications
You must be signed in to change notification settings - Fork 212
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
No c.flw or c.fsw in compressed testcases #289
Comments
Hmm, as far as I can tell, that is may be because there is a missing
support in Sail.
…On Tue, Oct 11, 2022 at 10:55 AM ssecatchseagate ***@***.***> wrote:
The floating point directories don't have any compressed instructions and
the compressed directories don't have any floating point instructions, so
the compressed floating point loads and stores aren't currently covered.
From 16.1 Overview:
" If the
C extension is implemented, the appropriate compressed floating-point load
and store instructions
must be provided whenever the relevant standard floating-point extension
(F and/or D) is also
implemented."
—
Reply to this email directly, view it on GitHub
<#289>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHPXVJXREXGGJVGX47ZWXKTWCWSZNANCNFSM6AAAAAARCQ3ZKY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
No, the support is in Sail; it appears that it was overlooked because it is shared by both F and C, and it looks like each group thought it was being dealt by the other. |
need a new set of coverpoints to get this up. |
To be precise, the following compressed floating-point loads and stores are not yet tested. c.flw, c.flwsp, c.fsw, c.fswsp, c.fld, c.fldsp, c.fsd, and c.fsdwp |
It has been a point of discussion for quite some time now. Even since these compressed instructions are categorised into different extensions (like Zca, Zcb, Zcf etc.). Its Sail support and ACTs are in development phase and still pending. This SoW (to provide Zc* extension support and its ACTs) was assigned to IITM at DevPartners. |
@ssecatch-w, Umer is correct that the work is being done by IITM. The DevPartners issue #6. This work is not yet done, but has recently begun being driven again by Anusha (@anuani21). Please read the issue for status and then reach out to Anusha and ensure that everything is being addressed or will be addressed. |
Tests have been added in #497 , so closing this one |
The floating point directories don't have any compressed instructions and the compressed directories don't have any floating point instructions, so the compressed floating point loads and stores aren't currently covered.
From 16.1 Overview:
" If the
C extension is implemented, the appropriate compressed floating-point load and store instructions
must be provided whenever the relevant standard floating-point extension (F and/or D) is also
implemented."
The text was updated successfully, but these errors were encountered: