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

packer: generalization of ignored cell types #121

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

yrabbit
Copy link
Collaborator

@yrabbit yrabbit commented Jul 22, 2022

  • Experiments with specialized routers can lead (in intermediate
    branches they already do) to the use of fake cell types, which are not
    encoded in any way, but only used to create the desired sequence of
    PIPs. To avoid adding a lot of ignored types, it is proposed to ignore
    types starting with DUMMY_.
  • Removed swearing on the unknown type MUX2_, but the type itself is
    left for the reason that although no action is required from the
    packer, it is still a real cell type.

Signed-off-by: YRabbit rabbit@yrabbit.cyou

* Experiments with specialized routers can lead (in intermediate
  branches they already do) to the use of fake cell types, which are not
  encoded in any way, but only used to create the desired sequence of
  PIPs. To avoid adding a lot of ignored types, it is proposed to ignore
  types starting with DUMMY_.
* Removed swearing on the unknown type MUX2_, but the type itself is
  left for the reason that although no action is required from the
  packer, it is still a real cell type.

Signed-off-by: YRabbit <rabbit@yrabbit.cyou>
@pepijndevos
Copy link
Member

Oh the CI still fails with the RAM error I should really really fix that... Other than that it seems fine.

@pepijndevos pepijndevos merged commit 634349a into YosysHQ:master Jul 27, 2022
@yrabbit yrabbit deleted the dummy-bels branch July 27, 2022 09:59
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