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.

Sign up for GitHub

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

Jump to bottom

C.SC and C.SCSP imm should be 64-bit aligned (RV32) #150

Merged

tariqkurd-repo merged 1 commit into riscv:main from James-Williams:James-Williams-rv32-csc-cscsp-imm

Mar 22, 2024

Contributor

James-Williams commented Mar 21, 2024 •

edited

Loading

C.SC and C.SCSP (RV32) store a 64-bit capability and should encode a 64-bit aligned offset.

C.LC (RV32) encodes offset[7:3]
C.LCSP (RV32) encodes offset[8:3]

C.SC (RV32) should likewise encode offset[7:3]
C.SCSP (RV32) should likewise encode offset[8:3]

This would match the RV64 RVC encodings which encode a 128-bit aligned offset for C.LC, CLCSP, C.SC, C.SCSP.


          C.SC and C.SCSP imm should be 64-bit aligned (RV32)

edbd9e0

Signed-off-by: James Williams <jamie.williams89@gmail.com>

tariqkurd-repo merged commit 68c8707 into riscv:main

3 checks passed

Collaborator

tariqkurd-repo commented Mar 22, 2024

Thaks for spotting this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet