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

show: use dot for wire aliases instead of BUF #3955

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

nakengelhardt
Copy link
Member

The "BUF" display is confusing as it is easy to misinterpret as a $_BUF_ cell.

Copy link
Member

@povik povik left a comment

Choose a reason for hiding this comment

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

The result even looks better to me.

@povik povik merged commit d510a5f into YosysHQ:master Sep 27, 2023
15 checks passed
@povik
Copy link
Member

povik commented Sep 27, 2023

Probably best to mention this in the changelog since people can be used to reading the show graphs a certain way.

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.

2 participants