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

fssr and frsr instructions aren't mentioned for F extension #419

Closed
laanwj opened this issue Jul 29, 2019 · 7 comments
Closed

fssr and frsr instructions aren't mentioned for F extension #419

laanwj opened this issue Jul 29, 2019 · 7 comments

Comments

@laanwj
Copy link

laanwj commented Jul 29, 2019

Just a documentation issue I think: The fssr and frsr instructions (which set and read the fcsr, respectively) aren't mentioned in the specification for the F extension. The names don't appear in the spec at all.

@laanwj laanwj changed the title fssr and frsr instructions aren't mentioned for F extensions fssr and frsr instructions aren't mentioned for F extension Jul 29, 2019
@aswaterman
Copy link
Member

aswaterman commented Jul 29, 2019 via email

@laanwj
Copy link
Author

laanwj commented Jul 29, 2019

Oh. Thanks!

I'm confused now, this came up in rust-embedded/riscv#157, there was the assumption that these were separate instructions, but if they map to the same thing then it's probably not necessary to mention them.

@Mel-Chen
Copy link

Hi! I also noticed this issue recently. Are frsr and fssr instructions obsolete, or renamed to frcsr and fscsr respectively? Thanks.

@aswaterman
Copy link
Member

aswaterman commented Jul 30, 2019 via email

@laanwj
Copy link
Author

laanwj commented Jul 30, 2019

Good to know!
RISC-V objdump as of version 2.32 still generates those aliases on disassembly.

@Mel-Chen
Copy link

Don't worried. I am preparing binutils patches to update objdump. :P

@laanwj
Copy link
Author

laanwj commented Jul 30, 2019

Thanks!

TheThirdOne added a commit to TheThirdOne/rars that referenced this issue Oct 14, 2019
These are just aliases for fscsr and frcsr respectively.
They are not mentioned in the specification as they are
outdated.

They were added because the move.s test uses them.

Information about them can be found at riscv/riscv-isa-manual#419
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

No branches or pull requests

3 participants