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.

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

Incorrect linking behavior for cm.jalt and cm.jt #369

Closed
silabs-hfegran opened this issue Dec 21, 2022 · 2 comments
Closed

Incorrect linking behavior for cm.jalt and cm.jt #369

silabs-hfegran opened this issue Dec 21, 2022 · 2 comments
Assignees
@silabs-oysteink
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation

Comments

@silabs-hfegran
Copy link
Contributor

Incorrect linking behavior for cm.jalt and cm.jt

Confirmed by inspection: RTL appear to

  • not treat cm.jt with index inside {[32:63]} as cm.jalt
  • treat cm.jalt with index inside {[32:63]} as cm.jt

This is due to a change in behavior from 0.70.x to 1.0.0-RC5.7, the previous threshold was 64.

image

Zc specification v.1.0.0-RC5.7:

3.5. cm.jalt
(...)

  • Note: For this encoding to decode as cm.jalt, index>=32, otherwise it decodes as cm.jt, see
    Jump via table.

3.4. cm.jt
(...)

  • For this encoding to decode as cm.jt, index<32, otherwise it decodes as cm.jalt, see
    Jump and link via table.
@silabs-hfegran silabs-hfegran added the Component:RTL For issues in the RTL (e.g. for files in the rtl directory) label Dec 21, 2022
@Silabs-ArjanB
Copy link
Contributor

Solved with openhwgroup/cv32e40x#763 (not merged to 40S yet)

@Silabs-ArjanB Silabs-ArjanB added the Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation label Jan 16, 2023
@Silabs-ArjanB
Copy link
Contributor

@silabs-hfegran Can you please close this if it has been fixed?

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

@silabs-oysteink silabs-oysteink

Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Silabs-ArjanB @silabs-oysteink @silabs-hfegran