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

Proposal: Add dependency to Zicsr #145

Open
a4lg opened this issue Sep 23, 2022 · 1 comment
Open

Proposal: Add dependency to Zicsr #145

a4lg opened this issue Sep 23, 2022 · 1 comment

Comments

@a4lg
Copy link

a4lg commented Sep 23, 2022

Proposal

I propose to add a note that the Zpn extension depends on the Zicsr extension.

Background

All legal combinations of RV32P/RV64P contain the Zpn extension, which contains RDOV and CLROV pseudoinstructions. They access vxsat CSR (URW, 0x9) and require the Zicsr extension.

This is basically the same issue report as riscvarchive/riscv-code-size-reduction#179 that is resolved by adding an extension dependency (ZcmtZicsr).

Notes

Although that OV flag is updated by some instructions in Zpsfoperand, I'm not sure whether Zpsfoperand should depend on the Zicsr (I think that ZpsfoperandZicsr is not necessary) because:

  1. Such access to OV flag is write-only
  2. Zpsfoperand does not contain instructions/pseudoinstructions that access OV flag via CSR.
@a4lg a4lg changed the title Add dependency to Zicsr Proposal: Add dependency to Zicsr Sep 28, 2022
@Kevin-Andes
Copy link
Collaborator

This is a very good suggestion and we should add dependency to Zicsr for Zpn.

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

2 participants