-
Notifications
You must be signed in to change notification settings - Fork 33
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
Fix endianess parameter descriptions #433
Conversation
Fixes #432 Signed-off-by: Derek Hower <[email protected]>
I think U-ext has the same issue: https://github.com/riscv-software-src/riscv-unified-db/blob/main/arch/ext/U.yaml#L23 |
This resolution looks good! |
I think this needs to be reopened in order to get U-mode fixed and get everything merged. |
Sorry, Paul -- (as a GitHub newbie,) I thought a new Issue needed to be opened, since U.yaml was a different file. |
No problem! (Just a bit surprising. ;-)
It can be used for both. Indeed, this is the vehicle for adding the changes, so the proposed change from Derek (and presumably an additional change from Derek for U.yaml) will appear here, where they can be reviewed, approved, and merged. That final act, merging, will automatically close this pull request. |
Signed-off-by: Derek Hower <[email protected]>
Signed-off-by: Derek Hower <[email protected]>
VU and VS had the same problem, updated them as well. |
Fixes #432