Skip to content

Conversation

@openroad-ci
Copy link
Collaborator

No description provided.


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
Copy link
Member

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.

Copy link
Member

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.

Copy link
Member

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?

Copy link
Member

Choose a reason for hiding this comment

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

sure

@maliberty
Copy link
Member

Is there any use of wrapping remaining or can we remove the whole mechanism to do wrapping?

@osamahammad21
Copy link
Member

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.

@maliberty
Copy link
Member

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?

@osamahammad21
Copy link
Member

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?

@maliberty
Copy link
Member

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.

@osamahammad21
Copy link
Member

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.
As for your original question if there are any remaining designs that do have wrapping, I see gf12/bp_single/config_mpl2.mk the only remaining test that have wrapping still. I am not sure if it's part of the secure CI or not. But other than that, none of the designs have wrapping anymore.

Signed-off-by: osamahammad21 <[email protected]>
@maliberty
Copy link
Member

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.

@maliberty
Copy link
Member

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.

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.

3 participants