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

cheri_interrupt_state should be cheriot #101

Open
davidchisnall opened this issue Jan 10, 2025 · 2 comments
Open

cheri_interrupt_state should be cheriot #101

davidchisnall opened this issue Jan 10, 2025 · 2 comments

Comments

@davidchisnall
Copy link

This attribute is specific to CHERIoT and very unlikely to exist on bigger systems. It is painful to implement on out-of-order cores.

We should keep the old name available as an alias, but issue a deprecated warning when it's used.

@resistor
Copy link
Collaborator

Relatedly, should we support some of the other attributes in the cheriot namespace as well? cheriot_compartment, for example? It's easy enough to have a second spelling.

@davidchisnall
Copy link
Author

davidchisnall commented Jan 13, 2025

Yes, I think that’s a good idea. The calling convention-related ones, in particular, are a good idea. We can always have a macro that expands to one of a selection based on target if some other CHERI platforms have them.

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

2 participants