Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

hdl._nir: AssignmentList: fix comb_edges_to #1514

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

rroohhh
Copy link
Contributor

@rroohhh rroohhh commented Sep 19, 2024

Only include the condition of a assignment in the comb edges if the assignment assigns to the bit in question.

Only include the condition of a assignment in the comb edges if the assignment assigns to the bit in question.
Copy link
Member

@wanda-phi wanda-phi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch, thanks

@whitequark whitequark added this pull request to the merge queue Sep 19, 2024
Merged via the queue into amaranth-lang:main with commit 8c5ed40 Sep 19, 2024
18 of 20 checks passed
@rroohhh rroohhh deleted the comb_cycle_assignment branch September 19, 2024 10:54
@whitequark
Copy link
Member

@rroohhh Please also prepare a backport PR to the 0.5 branch, and let me know if I should do a release.

@rroohhh
Copy link
Contributor Author

rroohhh commented Sep 19, 2024

Please find the backport PR at #1515.
I do not need a release, I just use git HEAD atm.

# for free to join this conversation on GitHub. Already have an account? # to comment
Development

Successfully merging this pull request may close these issues.

3 participants