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

Illegal condition for VWSLL operation #381

Open
Mingku-Chang opened this issue Jan 24, 2024 · 3 comments
Open

Illegal condition for VWSLL operation #381

Mingku-Chang opened this issue Jan 24, 2024 · 3 comments

Comments

@Mingku-Chang
Copy link

I guess that the VWSLL operation with (SEW == 64) is not allowed.
Vector Crypto follows a lot of regulations from Vector.
Therefore, results can't exceed 64-bit width.
I recommend adding some statements to show readers that Vector has such a regulation.

@nibrunieAtSi5
Copy link
Contributor

You are right vwsll with SEW=64-bit (vsew=3) would mean an effective element width of 128-bit (EEW=2*SEW) which is not currently supported so this behaviour is reserved.

Note that they are other possible restrictions, for example if ELEN=32 then SEW=32-bit is also reserved (Since Zvbb could be supported alongside Zve32x for example). In that case I am not sure a specific mention of SEW==64 really make sense since it is not the only possible limitation. We could (should ?) mention that this instructions inherits element width constraint from RVV.

@nibrunieAtSi5
Copy link
Contributor

@kdockser for your consideration

@wmat
Copy link
Contributor

wmat commented Mar 28, 2024

Is this recommendation still valid? If yes, please transfer issue to riscv-isa-manual repo or let me know to do it. Thanks.

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

3 participants