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

Add PMM field to *envcfg and mseccfg #1170

Merged
merged 1 commit into from
Dec 8, 2023
Merged

Add PMM field to *envcfg and mseccfg #1170

merged 1 commit into from
Dec 8, 2023

Conversation

aswaterman
Copy link
Member

@jhauser-us
Copy link
Collaborator

I thought the Architecture Review Committee wanted the name of the field to be PMM (Pointer Masking Mode), not PME.

@gfavor
Copy link
Collaborator

gfavor commented Dec 8, 2023 via email

@aswaterman
Copy link
Member Author

Thanks for the correction. I'll take this as approval modulo that change, so I'll merge shortly.

@aswaterman
Copy link
Member Author

@martinmaas please also update the pointer-masking spec to reflect the name PMM rather than PME.

@aswaterman aswaterman changed the title Add PME field to *envcfg and mseccfg Add PMM field to *envcfg and mseccfg Dec 8, 2023
@aswaterman aswaterman merged commit af51204 into main Dec 8, 2023
2 checks passed
@aswaterman aswaterman deleted the pme branch December 8, 2023 09:57
@martinmaas
Copy link
Contributor

This is now updated in the pointer masking spec as well:

riscv/riscv-j-extension@d6f29a6

@aswaterman
Copy link
Member Author

Thank you!

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

Successfully merging this pull request may close these issues.

4 participants