Skip to content

Odin report removed unused nodes #2077

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

Conversation

alirezazd
Copy link
Contributor

@alirezazd alirezazd commented Jun 28, 2022

  • Added a function that reports number of removed nodes by type after the cleanup stage.
  • Added a missing entry in odin_erro.h enum and regenerated the test results

Description

Related Issue

CAS-Atlantic/PR#36

#2008

Motivation and Context

It informs the user of number of removed logic elements by type.

How Has This Been Tested?

Types of changes

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

alirezazd and others added 3 commits June 27, 2022 10:34
Fixed a missing enum entry in odin_error.h

Signed-off-by: Alireza Azadi <[email protected]>
…t variable as UNUSED_NODE_TYPE

	- add more comments and fix code style

Signed-off-by: Seyed Alireza Damghani <[email protected]>
@github-actions github-actions bot added the Odin Odin II Logic Synthesis Tool: Unsorted item label Jun 28, 2022
@alirezazd
Copy link
Contributor Author

@sdamghan FYI, Ready to merge.

@sdamghan
Copy link
Member

Thanks @alirezazd

@sdamghan sdamghan merged commit 6c93392 into verilog-to-routing:master Jun 28, 2022
@alirezazd alirezazd deleted the odin_check_netlist_discontinuity branch June 28, 2022 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Odin Odin II Logic Synthesis Tool: Unsorted item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants