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
Add check for clock type in CoreIR compilation #289
Conversation
Similar to what we do for AysncReset types, we make a special case to be able to wire constants to clocks.
Pull Request Test Coverage Report for Build 1110
💛 - Coveralls |
Looks good, any chance we can get a little test for this? I think the code in the issue should work. |
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
tests/test_type/test_clock.py
Outdated
Reset, ResetType, ResetKind, \ | ||
Enable, EnableType, EnableKind | ||
Reset, ResetType, ResetKind, reset \ | ||
Enable, EnableType, EnableKind, enable \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing end commas here I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made the change via the github web ui, i'll wait for the build
Similar to what we do for AysncReset types, we make a special case to be
able to wire constants to clocks.