-
Notifications
You must be signed in to change notification settings - Fork 407
Remove gf12 wrappers #2638
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
base: master
Are you sure you want to change the base?
Remove gf12 wrappers #2638
Conversation
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
flow/designs/gf12/ca53/config.mk
Outdated
|
|
||
| export ADDITIONAL_LEFS += $(PLATFORM_DIR)/lef/sc9mcpp84_12lp_base_lvt_c14.lef | ||
| export ADDITIONAL_LIBS += $(PLATFORM_DIR)/lib/sc9mcpp84_12lp_base_lvt_c14_tt_nominal_max_0p80v_25c.lib | ||
| export ADDITIONAL_GDS += $(PLATFORM_DIR)/gds/sc9mcpp84_12lp_base_lvt_c14.gds2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are the lvt cells removed (but not their gds)? This seems unrelated to wrapping.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are not removed, they are just added later in the config file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that case why not move the gds too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure
|
Is there any use of wrapping remaining or can we remove the whole mechanism to do wrapping? |
Not to the detailed router. Not sure about other parts of the flow. |
Wrapping is not drt specific. If you changed the config.mk you affected all steps already. I was more asking if there are any designs still with wrapping? |
|
To make sure I understand correctly, if I remove all the wrap_lefs, wrap_defs and additional_gds files from the configurations(not just move them to additionals), then no wrapping would happen and nothing should break, right? |
How is removing them different from moving them? They have to appear in additional or they wouldn't exist at all in the config. |
Sorry got confused. I thought that these files are used for wrapping only and didn't understand that they hold the actual macros being used originally in the design. |
Signed-off-by: osamahammad21 <[email protected]>
|
I think gf12/bp_single/config_mpl2.mk is an experiment to use mpl2 rather than manual macro placement. It isn't in the CI. Please make a follow on PR to remove the wrapping support from the flow itself. |
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
Signed-off-by: Osama <[email protected]>
|
This was approved but never got merged. I see it now has a conflict. It would be nice to update this so we can finish removing the wrappers if possible. Not urgent. |
No description provided.