-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added requirement for 4B accesses to be atomic if 4B aligned #43
Conversation
LGTM |
dc31b7d
to
c798edc
Compare
zilsd.adoc
Outdated
*** If the effective address is 4B aligned: | ||
**** The bytes are grouped into word accesses. | ||
**** The words may be stored in any order. | ||
**** The words may be grouped into larger accesses. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets tighten this a bit. The words may be grouped into a doubleword. Just to avoid someone creating a 5B + 3B grouping.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that was unintended copy paste error. fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should also be fixed for the stores.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry, i did fix it but failed to push. i think it looks fine now
No description provided.