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

The tail element does not follow the RVV SPEC requirements. #303

Open
TimLee-123 opened this issue May 22, 2024 · 1 comment
Open

The tail element does not follow the RVV SPEC requirements. #303

TimLee-123 opened this issue May 22, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@TimLee-123
Copy link

Issue

I have fixed the bug in issue #298. During verification, I discovered that the design has issues with handling tail elements. The waveform illustrates that the instruction vadd.vx (sew = 32, vl = 12, vd = v23) writes 64'h2000_0004_2000_0004 into lane0_bank0 of v23. The subsequent instruction, vid (sew = 32, vl = 2, vd = v23), when executed and written back, should theoretically produce 64'h2000_0004_0000_0000 or 64'hffff_ffff_0000_0000 according to RVV SPEC, section 3.4.3. However, since the design does not adhere to this specification, a bug occurs.

Supplementary Pictures

image

image

image

@mp-17 mp-17 added the bug Something isn't working label Jun 14, 2024
@mp-17
Copy link
Collaborator

mp-17 commented Nov 25, 2024

This should have been fixed with the last MASKU refactoring in this PR. Let me know if the bug is still around. If you want to send me a code snippet to reproduce the bug, I can also check :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants