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

Eliminate redundant statement about AMO's stored value availability #1024

Merged
merged 1 commit into from
May 15, 2024

Conversation

en-sc
Copy link
Contributor

@en-sc en-sc commented Apr 26, 2024

It is covered by

Whether data store triggers match on AMOs is UNSPECIFIED.

It is covered by
> Whether data store triggers match on AMOs is UNSPECIFIED.
Copy link
Collaborator

@rtwfroody rtwfroody left a comment

Choose a reason for hiding this comment

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

This is more concise and just as clear.

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.

This looks good. There's no normative change but it's clearer and the formatting is more consistent (rather than having some things in bullets and some not in bullets).

@rtwfroody
Copy link
Collaborator

Now that we're frozen, we are still allowed to make "minimal, non-substantive" changes (https://docs.google.com/document/d/1Au3veNdNJQKPq-oiQRKTzdgmM72FDaqZOKeH7sOnG04/edit#heading=h.fjjoh42d729w). This qualifies.

@rtwfroody rtwfroody merged commit 99450e5 into riscv:main May 15, 2024
1 check passed
@en-sc en-sc deleted the en-sc/amo-stores branch May 17, 2024 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants