-
Notifications
You must be signed in to change notification settings - Fork 218
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
Pseudo-instruction JR #43
Comments
I thought I copied all of the specified psuedo instructions, but it appears I was wrong. This is handled at Lines 105 to 106 in 6afb286
Its a two line addition. I can push it up in a few hours. |
@TheThirdOne Would it be possible to make a new release for RARS? I just noticed that the latest release does not include this bug fix but the pre-release does. We would have a group of students download RARS soon (some already did) and we would like for this fix to be included. |
@rmnattas, I definitely could make a new release. The reason I made the continuously updated per-release is to make it easy to get access to the latest bugfixes. The main releases are then intended to each have significant developments over the last. There have been a few small psuedo-op improvemnts and progress on two outstanding issues since v1.3, but thats a pretty small set. What is the reason you are uncomfortable with using the continuous release? If possible, I would like to improve the continuous release system so it could satisfy this kind of request. |
@TheThirdOne We are providing the repo link to students before the terms begin and they mostly will be downloading the latest release instead of the pre-release, and we would prefer if there's a version number (i.e. 1.3.1) to specify and work on. |
v1.3.1 now exists. |
This is related to TheThirdOne#43.
In page 110 of the RISC-V Specifications the format for the pseudo-instruction
jr
isjr rs
.When I used
jr ra
in RARS I got the following error:Too few or incorrectly formatted operands. Expected: jr t0, -100
I think keeping the format
jr t0, -100
would be beneficial but also supporting the official one is important.Where does RARS handle pseudo-instruction in the source code?
The text was updated successfully, but these errors were encountered: