Skip to content

Updated SRAM sizes for asap7 cva6#3196

Merged
maliberty merged 2 commits into
The-OpenROAD-Project:masterfrom
jeffng-or:cva6-sram-256-256
May 29, 2025
Merged

Updated SRAM sizes for asap7 cva6#3196
maliberty merged 2 commits into
The-OpenROAD-Project:masterfrom
jeffng-or:cva6-sram-256-256

Conversation

@jeffng-or

Copy link
Copy Markdown
Contributor

Updated ram size to 256x256 to prevent RAMs from being optimized out in flat synthesis mode

@jeffng-or jeffng-or requested review from maliberty and povik May 28, 2025 18:26
Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
@jeffng-or jeffng-or force-pushed the cva6-sram-256-256 branch from d51e1f1 to 5917742 Compare May 28, 2025 18:32
@maliberty maliberty enabled auto-merge May 28, 2025 20:06
Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
auto-merge was automatically disabled May 28, 2025 22:50

Head branch was pushed to by a user without write access

@jeffng-or

Copy link
Copy Markdown
Contributor Author

designs/asap7/cva6/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 16477.72 40692.1 Failing
placeopt__design__instance__area 19790 45043 Failing
placeopt__design__instance__count__stdcell 130789 164118 Failing
finish__design__instance__area 20112 45315 Failing

@maliberty

Copy link
Copy Markdown
Member

I guess the jump in area is because we are no longer getting false dead logic elimination?

@maliberty maliberty enabled auto-merge May 28, 2025 22:58
@jeffng-or

Copy link
Copy Markdown
Contributor Author

I guess the jump in area is because we are no longer getting false dead logic elimination?

That and the RAMs are bigger. They were previously 256x32 and now they're 256x256.

@maliberty maliberty merged commit 19538fb into The-OpenROAD-Project:master May 29, 2025
7 checks passed
@jeffng-or jeffng-or deleted the cva6-sram-256-256 branch August 22, 2025 16:13
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.

2 participants