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.

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

[rtl] CPU: optimizations and cleanup #462

Merged
merged 6 commits into from Dec 21, 2022
Merged

[rtl] CPU: optimizations and cleanup #462

merged 6 commits into from Dec 21, 2022

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Dec 20, 2022

  • remove reset to don't care ('-')
  • connect FIFO reset
  • optimization in branch detection logic (improved area and timing)
  • optimization in CSR access check logic (improved area and timing)
  • minor code edits and cleanups
  • closed further illegal instruction encoding hole (CSR access instruction with funct3 = 100)

@stnolting stnolting added enhancement New feature or request HW hardware-related coding-style Related to the HW/SW coding style labels Dec 20, 2022
@stnolting stnolting self-assigned this Dec 20, 2022
@stnolting stnolting added optimization Make things faster, smaller and more efficient and removed enhancement New feature or request labels Dec 20, 2022
* connect reset of IPB FIFO
* optimize branch condition logic
* remove reset-to-don't-care
* optimize CSR access check logic
* close further illegal instruction encoding hole
* minor code cleanups
@stnolting stnolting marked this pull request as ready for review December 21, 2022 05:41
@stnolting stnolting merged commit aec195f into main Dec 21, 2022
@stnolting stnolting deleted the cpu_rtl_updates branch December 21, 2022 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coding-style Related to the HW/SW coding style HW hardware-related optimization Make things faster, smaller and more efficient
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant