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

Extension names need to be finalized #535

Open
arichardson opened this issue Feb 13, 2025 · 2 comments
Open

Extension names need to be finalized #535

arichardson opened this issue Feb 13, 2025 · 2 comments

Comments

@arichardson
Copy link
Collaborator

The current extension names are not valid since Zc* is currently reserved for extensions that affect compressed instructions. The names in the ISA document are placeholders for now (unless we decide that Zc* only refers to compressed instructions if it does not start with Zcheri*).

@jrtc27
Copy link
Collaborator

jrtc27 commented Feb 13, 2025

Even aside from that, Zcheripte is a privileged architecture extension so should be Sfoo. Since it specifically relates to translation, it should even be Svfoo. On top of that, Svcheripte (or Svcheri once you remove the wholly-redundant "pte" suffix) is too generic a name for adding (U)CRG. Something like "Svucrg" is probably the right name for it, which is completely different to what it's currently called.

@arichardson
Copy link
Collaborator Author

Even aside from that, Zcheripte is a privileged architecture extension so should be Sfoo. Since it specifically relates to translation, it should even be Svfoo. On top of that, Svcheripte (or Svcheri once you remove the wholly-redundant "pte" suffix) is too generic a name for adding (U)CRG. Something like "Svucrg" is probably the right name for it, which is completely different to what it's currently called.

Agreed, I think we should proactively use Svucrg which is clearly better than the current placeholder. Will open a PR shortly.

arichardson added a commit that referenced this issue Feb 14, 2025
While this is still a placeholder, we at least use the correct prefix now.

See #535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants