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

[Cider] Clock propagation #2368

Merged
merged 33 commits into from
Dec 4, 2024
Merged

[Cider] Clock propagation #2368

merged 33 commits into from
Dec 4, 2024

Conversation

EclecticGriffin
Copy link
Collaborator

@EclecticGriffin EclecticGriffin commented Dec 4, 2024

This somewhat chunky PR includes the logic needed to propagate clocks through combinational logic and amends the read checks to defer until the value is used in a non-combinational context. The continuous assignments now run under a single thread which is never synchronized, so attempts to write a register in continuous logic will always result in a race when used outside continuous logic.

There's some additional refactoring and minor changes to the primitive interface. Additionally, a single step can now advance past multiple control nodes, which should reduce some of the repetition when using the debugger

@EclecticGriffin EclecticGriffin merged commit 445e174 into main Dec 4, 2024
18 checks passed
@EclecticGriffin EclecticGriffin deleted the cider/clock-comb-prop branch December 4, 2024 19:58
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

Successfully merging this pull request may close these issues.

1 participant