Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Minor bug fix in compressed immediate decoding #74

Open
wants to merge 15 commits into
base: dev
Choose a base branch
from
Open

Minor bug fix in compressed immediate decoding #74

wants to merge 15 commits into from

Conversation

mahendraVamshi
Copy link
Contributor

[0.18.1] - 2023-07-31

  • Minor fix to compressed immediate decoding

@mahendraVamshi
Copy link
Contributor Author

Hi @pawks,
This is a minor bug fix from previous PR #73

@mahendraVamshi
Update rvopcodesdecoder.py
29599e2 
fixed the rs1,rs2,rs3 and rd names for compressed instructions

Signed-off-by: Mahendra Vamshi A <[email protected]>
@neelgala
Copy link
Collaborator

@mahendraVamshi can you elaborate more on what corner case this PR fixes ?

@mahendraVamshi
Copy link
Contributor Author

@neelgala
instr: 62510 addr: 0x800014d2 instr_name: c.sdsp rs2: (11, 'x') imm: -24
This instruction uses above immideate field and the above PR fixes the imm field for this instruction
c.sdsp instructuions uses the fixed imm field

@neelgala
Copy link
Collaborator

@mahendraVamshi please change your target branch to dev. Remove all versioning changes in setup* and init.py. Also the changelog entry must be under WIP-DEV section of the CHANGELOG.md file.

mahendraVamshi and others added 11 commits September 21, 2023 06:12
@mahendraVamshi
Revert "Update rvopcodesdecoder.py"
a91da56 
This reverts commit 29599e2.
@mahendraVamshi
Update spike.py
73acdc1 
fixed regex pattern

Signed-off-by: Mahendra Vamshi A <[email protected]>
@mahendraVamshi
Update rvopcodesdecoder.py
eaa6404 
Signed-off-by: Mahendra Vamshi A <[email protected]>
@mahendraVamshi
Update rvopcodesdecoder.py
3e17fc6 
Signed-off-by: Mahendra Vamshi A <[email protected]>
@mahendraVamshi
experiment
b43aacd
@mahendraVamshi
exp
abbe92e
@mahendraVamshi
exp
93a5b4a
@mahendraVamshi
exp
5deaf26
@mahendraVamshi
revert back
e2f8e62
@mahendraVamshi
Update rvopcodesdecoder.py
2b5343e 
Signed-off-by: Mahendra Vamshi A <[email protected]>
@mahendraVamshi
Update rvopcodesdecoder.py
d592747 
Signed-off-by: Mahendra Vamshi A <[email protected]>
@jamesbeyond jamesbeyond changed the base branch from master to dev May 28, 2024 17:20
@jamesbeyond
Copy link
Collaborator

Dear @mahendraVamshi

The CTG and ISAC Repo had been moved to riscv-arch-test via PR
riscv-non-isa/riscv-arch-test#495

If you think this PR is important, and need to me merged on to risc-v CTG or ISAC, please re-submit your PR against riscv-arch-test

This repository shall be archived in about one week time .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mahendraVamshi @neelgala @jamesbeyond