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

AR: Remove comment about privspec/CSR behavior #874

Merged
merged 1 commit into from Sep 14, 2023
Merged

Conversation

timsifive
Copy link
Contributor

From Greg: "... why is this non-normative comment even necessary? It's directly consistent with 1.12, and it's compatible with 1.13 which allows for trapping and just doesn't require it. In other words, wrt 1.13, this trapping mandate by Debug 1.0 doesn't conflict with Priv 1.13. And that allows Debug 1.0 to come along and tighten what is allowed by 1.13."

From Greg: "... why is this non-normative comment even necessary?  It's
directly consistent with 1.12, and it's compatible with 1.13 which
allows for trapping and just doesn't require it.  In other words, wrt
1.13, this trapping mandate by Debug 1.0 doesn't conflict with Priv
1.13.  And that allows Debug 1.0 to come along and tighten what is
allowed by 1.13."
Copy link
Collaborator

@pdonahue-ventana pdonahue-ventana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Back to the original.

@timsifive timsifive merged commit fdfbbaf into master Sep 14, 2023
1 check passed
@timsifive timsifive deleted the ar_unimplemented branch September 14, 2023 18:08
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.

None yet

2 participants