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

Add cluster-level synthesis file lists for tapeout #19

Merged
merged 8 commits into from
Aug 29, 2024
Merged

Conversation

rgantonio
Copy link

@rgantonio rgantonio commented Aug 27, 2024

This PR adds the generation for file lists per cluster.

Major TODO:

  • Create and add makefile for tapeout target
  • Create and add into the occamygen and occamy the call functions for generating the filelists
  • Modify top-level Makefile
  • Re-base and modify on top of PR hemaia util #20

@rgantonio rgantonio added the enhancement New feature or request label Aug 27, 2024
@rgantonio rgantonio self-assigned this Aug 27, 2024
@rgantonio
Copy link
Author

@IveanEx and @Konste11ation plez approve hahahaha so I can move forward to synthesis!

@IveanEx
Copy link

IveanEx commented Aug 27, 2024

I just think about if you can avoid modifying occamygen, as the the filelist should not be the work of occamygen...

Copy link

@IveanEx IveanEx left a comment

Choose a reason for hiding this comment

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

Wait for another PR on hemaia_util.py

@rgantonio
Copy link
Author

I just think about if you can avoid modifying occamygen, as the the filelist should not be the work of occamygen...

OK I shall wait for it and make the necessary changes then.

@rgantonio rgantonio requested a review from IveanEx August 29, 2024 06:07
@rgantonio
Copy link
Author

Bump! @IveanEx and @Konste11ation here are the modifications to utilize the Hemaia util.

Copy link

@IveanEx IveanEx left a comment

Choose a reason for hiding this comment

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

Goodjob! It is much cleaner!

@rgantonio
Copy link
Author

@IveanEx thanks for the fix too! Will push this now 😄

@rgantonio rgantonio merged commit 7efe8db into main Aug 29, 2024
2 checks passed
@rgantonio rgantonio deleted the ry/add-syn-gen branch August 29, 2024 08:41
Konste11ation pushed a commit that referenced this pull request Aug 31, 2024
* make: add gen-syn-flist target

* util: add occamy related generation

* make: add tapeout makefile

* util: add functionality for flist gen

* util: refactor flist gen

* make: modify target recipe

* util: fmt

* Improve hemaia util

---------

Co-authored-by: Yunhao Deng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants