Skip to content

Modified func3&7, register extraction block in Intr_decode #21

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Modified func3&7, register extraction block in Intr_decode #21

wants to merge 1 commit into from

Conversation

brettyang02
Copy link
Contributor

to enable all instructions in the table:

  • Instruction_Decode.v - func3 & func7 extration block
    • the current func3 & func7 extraction is for few instructions only, and needs to be updated
    • LW and I type should go in the same if clause, as LW type also has different func3
    • changed IType to IType_logic and change LWType to IType_load
    • added if clause for SType, BType
  • Instruction_Decode.v - rd, rs1 & rs2 extration block
    • need to add UType (can be in the same if clause as JType)
  • Instruction_Decode.v - Imm extration block
    • added UType
  • ControlFSM.v
    • used MACROs in next state logic block

@brettyang02
Modified func3&7, register extraction block
4e60a2c 
to enable all instructions in the table:

- Instruction_Decode.v - func3 & func7 extration block
    - the current func3 & func7 extraction is for few instructions only, and needs to be updated
    - LW and I type should go in the same if clause, as LW type also has different func3
    - changed IType to IType_logic and change LWType to IType_load
    - added if clause for SType, BType
- Instruction_Decode.v - rd, rs1 & rs2 extration block
    - need to add UType (can be in the same if clause as JType)
- Instruction_Decode.v - Imm extration block
    - added UType
- ControlFSM.v
    - used MACROs in next state logic block
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.
1 participant
@brettyang02