-
Notifications
You must be signed in to change notification settings - Fork 897
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
Adjust keep_hierarchy
behavior
#4706
Conversation
There's still a bit of a disconnect with the |
When I built this I experimented with also running opt in the flow and it didn't make much of a difference vs just techmapping, so I think this may not be as much of a problem? Anyway, it's just a knob for the user to twiddle, and users with an engineering background (the core audience) usually like that kind of arbitrary control, even if to a fault |
For instance numbers I expect opt isn’t as important as the abc transformations and mapping to technology cells |
What are the reasons/motivation for this change?
This makes two changes to
keep_hierarchy -min_cost
to make it better suited to the task of partially preserving hierarchy for macro placement:keep_hierarchy
towards the upper module's cost.gate_cost_equivalent
attribute as a way of specifying cost on blackboxes.