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

Case statements in rtl for mux #20

Open
lighteningfingers opened this issue May 5, 2024 · 1 comment
Open

Case statements in rtl for mux #20

lighteningfingers opened this issue May 5, 2024 · 1 comment

Comments

@lighteningfingers
Copy link

Just a comment on your case statements in your rtl.

for mutually exclusive case statements (say an opcode decode), a mux implementation is preferred over a priority encoder, for multiple reasons (sta, area etc etc).

i suggest refactoring your code for any mux implementations and considering “unique case” instead of case.
Note: by default in sv, a case statement is a priority encoder (see Cummings et al)

@adam-maj
Copy link
Owner

adam-maj commented May 6, 2024

@lighteningfingers - Thanks for the feedback - makes complete sense. I'm likely not actively updating this project, but I'll be looking at PRs for anyone who puts things up - feel free to put up a PR with the changes if you end up wanting to!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants