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.

Sign up for GitHub

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

Jump to bottom

Add more instance count-related metrics #563

Merged
merged 18 commits into from
Oct 8, 2024
Merged

Add more instance count-related metrics #563

merged 18 commits into from
Oct 8, 2024

Conversation

donn
Copy link
Member

@donn donn commented Sep 26, 2024 •
edited
Loading

Steps

  • OpenROAD.STA*

    • New report clock.rpt created with information about each clock in a specific domain

  • OpenROAD.*

    • All steps that modify views now update design cell metrics using OpenROAD's report_design_area_metrics

  • Created Odb.CellFrequencyTables

    • Creates a number of tables to show the cell frequencies by:
      • Cells
      • Buffer cells only
      • Cell Function
      • SCL

Flows

  • Classic, VHDLClassic
    • Added Odb.CellFrequencyTables after OpenROAD.FillInsertion

Tool Updates

  • OpenROAD -> bbe940134bddf836894bfd1fe02153f4a38f8ae5
    • OpenSTA -> 20925bb00965c1199c45aca0318c2baeb4042c5a
    • Removed "stable" version of OpenSTA

Misc Bugfixes/Enhancements

  • openlane.common.metrics
    • aggregate_metrics(): Added support for aggregation of N-modifier levels

Resolves #254
Resolves #547
Depends on efabless/openlane2-step-unit-tests#38
Depends on efabless/openlane2-ci-designs#33

kareefardi and others added 9 commits July 18, 2024 16:23
@kareefardi
Add new metrics
7c6da72 
ignore runs in examples

Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
Remove script
59b5d4c 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
Remove cts_count
6c527f5 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
Add step to count buffers
e1e9774 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
add a number of new metrics
56a529c 
enhance sta clock report

Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
Merge branch 'dev' into cell-count-metric
78e0337 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
updates:
cf7b3a6 
update openroad
update opensta
more robust way of finding cell types
remove decap count

Signed-off-by: Kareem Farid <[email protected]>
@donn
Merge remote-tracking branch 'origin/dev' into cell-count-metric
c496e30
@donn
Use modifiers for design__instance__count__stdcell
a820b3d
@openlane-bot
Copy link
Collaborator

openlane-bot commented Sep 26, 2024 •
edited
Loading

Failed to create report:

Metrics not found for commit: f18bb22e894bbd9fc5868211602cb049bc447efe.

@donn donn marked this pull request as ready for review September 29, 2024 10:39
@donn donn mentioned this pull request Sep 29, 2024
Merged
@kareefardi kareefardi mentioned this pull request Oct 8, 2024
Open
@donn donn mentioned this pull request Oct 8, 2024
Closed
@donn donn requested a review from kareefardi October 8, 2024 09:07
@donn donn merged commit f18bb22 into dev Oct 8, 2024
4 of 5 checks passed
@donn donn deleted the cell-count-metric branch October 8, 2024 11:33
donn added a commit that referenced this pull request Oct 20, 2024
@donn
Add #563 (missing)
c4c15e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kareefardi kareefardi kareefardi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@donn @openlane-bot @kareefardi