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

Rewrite description of FCSR. #289

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rewrite description of FCSR. #289

wants to merge 1 commit into from

Conversation

kito-cheng
Copy link
Collaborator

Try to rewrite description of FCSR, remove "thread storage duration", and reference of specific language standard, and also try to describe properties for different fields.

Description of frm are borrowed some word from the C lang spec[1].
And fflags might be changed by floating point operations or any function call, and no any promise of the value on the function entry.
Description of other bits are reference unpriv spec.

[1] ISO/IEC 9899:2011: 7.6.3

— a function call does not alter its caller’s floating-point control modes, clear its caller’s
floating-point status flags, nor depend on the state of its caller’s floating-point status
flags unless the function is so documented;

- Remove "thread storage duration".
- Describe properties for different fields.
@kito-cheng
Copy link
Collaborator Author

@nick-knight I don't know why I can add @aswaterman to reviewer but can't add you, so tag you manually :P

@nick-knight
Copy link
Contributor

@kito-cheng: I don't have privilege to review PRs in riscv-non-isa repos like this one. An admin will have to extend my privileges. (I would argue that this privilege should be a default, but I don't fully understand the ramifications.)

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.

2 participants