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 sideleg CSR might be deleted #529

Closed
KotorinMinami opened this issue Aug 27, 2024 · 2 comments
Closed

The sideleg CSR might be deleted #529

KotorinMinami opened this issue Aug 27, 2024 · 2 comments

Comments

@KotorinMinami
Copy link
Contributor

Recently, I reviewed the latest specification and found that the "sideleg" register no longer exists in the CSR table. Upon further investigation, it appears to have been removed. However, in the sail model, this register and its encoding still exist. Should this be removed to avoid potential issues with illegal encodings in the sail?

@jrtc27
Copy link
Collaborator

jrtc27 commented Aug 27, 2024

It's gated on Ext_N, which there isn't a way to enable unless you patch the source. The only issue that would arise is if the encoding is reused for another CSR. Though it would probably be wise to remove the implementation given the obsolete draft was removed outright from the riscv-isa-manual repo.

@jordancarlin
Copy link
Contributor

Closed by #584

@Timmmm Timmmm closed this as completed Oct 16, 2024
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

4 participants