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

Decoding pseudo-code clarification/fix #309

Open
tony-cole opened this issue Jul 2, 2024 · 1 comment
Open

Decoding pseudo-code clarification/fix #309

tony-cole opened this issue Jul 2, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request future enhancement

Comments

@tony-cole
Copy link

Re: riscv-cheri-v0.8.2.pdf the Decoding pseudo-code on page 14.

For the calculations of ct and cb to be correct, R needs to be bounded to MW bits, as follows:
R[MW-1:0] = B - 2^MW-2^

and note that the comparisons in Tables 8 & 9 are unsigned.

@andresag01
Copy link
Collaborator

@tony-cole : This part of the spec is only a place-holder and the current notation is not terribly well-defined as you point out. This is because the text is temporary and will eventually be replaced with auto-generated fragments of SAIL extracted from the open-source RISC-V CHERI SAIL model.

We should keep it like this for now and revisit when the SAIL is available. Note that we've had similar comments in the past.

@andresag01 andresag01 added documentation Improvements or additions to documentation enhancement New feature or request future enhancement labels Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request future enhancement
Projects
None yet
Development

No branches or pull requests

2 participants