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.

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

🧹 [rtl] CPU frontend cleanup #299

Merged
merged 10 commits into from
Apr 11, 2022
Merged

🧹 [rtl] CPU frontend cleanup #299

merged 10 commits into from
Apr 11, 2022

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Apr 11, 2022 •
edited
Loading

This a rather large cleanup and optimization PR:

  • remove instruction fetch from CPU's bus unit (this is now a load/store unit processing only data memory accesses)
  • cleanup instruction fetch interface; there were a lot of redundancies in the bus unit's instruction fetch logic
  • optimize CPU front end logic (instruction fetch) reducing area costs

neorv32_cpu

@stnolting stnolting added the HW Hardware-related label Apr 11, 2022
@stnolting stnolting self-assigned this Apr 11, 2022
@stnolting stnolting changed the title [rtl] 🧹 CPU frontend cleanup 🧹 [rtl] CPU frontend cleanup Apr 11, 2022
@stnolting stnolting marked this pull request as ready for review April 11, 2022 04:57
@stnolting stnolting merged commit e16ec73 into main Apr 11, 2022
@stnolting stnolting deleted the frontend_cleanup branch April 11, 2022 18:24
@stnolting stnolting added the optimization Make things faster, smaller and more efficient label Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@stnolting stnolting

Labels
HW Hardware-related optimization Make things faster, smaller and more efficient
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stnolting