You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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)
The text was updated successfully, but these errors were encountered:
@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!
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)
The text was updated successfully, but these errors were encountered: