Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

jit: bug if the registers of the source and linkage registers of JALR, CJALR instructions are same #472

Closed
vacantron opened this issue Aug 3, 2024 · 0 comments
Closed

jit: bug if the registers of the source and linkage registers of JALR, CJALR instructions are same #472

vacantron opened this issue Aug 3, 2024 · 0 comments
Assignees
@vacantron
Labels
bug Something isn't working
Milestone
release-2024.1

Comments

@vacantron
Copy link
Collaborator

Consider the following assembly to call a far-way function,

la    ra,<some label>
jalr  ra,0(0)

It would not work as expectation.
@vacantron vacantron added the bug Something isn't working label Aug 3, 2024
@vacantron vacantron self-assigned this Aug 3, 2024
@jserv jserv added this to the release-2024.1 milestone Aug 7, 2024
vacantron added a commit to vacantron/rv32emu that referenced this issue Aug 21, 2024
@vacantron
Fix unexpected "JALR" behavior in JIT
b6b65a4 
The register which stores the indirect address needs to be loaded first
to avoid being overriden when the operands are same in "JALR", "C.JALR".

Fix sysprog21#472
@jserv jserv closed this as completed in 559dffe Aug 21, 2024
vestata pushed a commit to vestata/rv32emu that referenced this issue Jan 24, 2025
@vacantron
Fix unexpected "JALR" behavior in JIT
09d5d00 
The register which stores the indirect address needs to be loaded first
to avoid being overriden when the operands are same in "JALR", "C.JALR".

Fix sysprog21#472
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@vacantron vacantron

Labels
bug Something isn't working
Projects
None yet
Milestone
release-2024.1
Development

No branches or pull requests
2 participants
@jserv @vacantron