Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

priv-1.13: clarify that MXLEN >= SXLEN; constrain SXLEN >= UXLEN #1028

Merged
merged 2 commits into from
Jul 21, 2023

Conversation

aswaterman
Copy link
Member

@gfavor FYI

@jhauser-us please check my wording. It's a little obtuse because I was trying to constrain the set of legal values for SXL's WARL purposes.

@aswaterman aswaterman requested review from jhauser-us and gfavor April 28, 2023 06:39
@jhauser-us
Copy link
Collaborator

I would probably have:

If S-mode is implemented, the set of legal values that the UXL field may assume excludes those that would cause UXLEN to be greater than SXLEN.

Because: (1) Is SXLEN defined without S-mode? (2) To me it seems more natural to have UXLEN bounded by SXLEN.

@aswaterman
Copy link
Member Author

Good points. PR updated with your proposed wording.

Greg approved via email, but I'll wait to merge until we have the chance to discuss this with the ARC.

As a consequence, this clarifies that MXLEN >= SXLEN.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants