Skip to content

Log hashes per step#3299

Merged
maliberty merged 3 commits into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:log-hashes-per-step
Jul 8, 2025
Merged

Log hashes per step#3299
maliberty merged 3 commits into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:log-hashes-per-step

Conversation

@oharboe

@oharboe oharboe commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

Review only the most recent commit, builds on #3297

Example output:

$ make do-2_1_floorplan
[deleted]
Design area 650 u^2 57% utilization.
Elapsed time: 0:00.28[h:]min:sec. CPU time: user 0.29 sys 0.12 (147%). Peak memory: 101020KB.
Log                        Elapsed/s Peak Memory/MB  sha1sum .odb [0:20)
2_1_floorplan                      0             98 50eac5062cd153c214bc

In the bazel-orfs use-case, the .odb files are not all present in the finish step, so log as we go along in addition to the finish step.

A little bit more verbose than I could have wished for, but it does the job. Refine later, possibly.

oharboe added 3 commits July 7, 2025 21:51
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
In the bazel-orfs use-case, the .odb files are not all present
in the finish step, so log as we go along in addition to the finish
step.

A little bit more verbose than I could have wished for, but it does
the job. Refine later, possibly.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe oharboe force-pushed the log-hashes-per-step branch from ac6bb6c to c537a7e Compare July 7, 2025 21:04
@maliberty maliberty enabled auto-merge July 7, 2025 21:45
@oharboe

oharboe commented Jul 8, 2025

Copy link
Copy Markdown
Collaborator Author

@maliberty merge manually? asap7 cva6 timeouts...

@maliberty maliberty disabled auto-merge July 8, 2025 03:44
@maliberty maliberty merged commit 0ae5fce into The-OpenROAD-Project:master Jul 8, 2025
8 of 9 checks passed
@oharboe oharboe deleted the log-hashes-per-step branch July 8, 2025 06:32
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