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.

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

consider state-dependent leakage during sizing #6791

Conversation

openroad-ci
Copy link
Collaborator

No description provided.

Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Cho Moon <cmoon@precisioninno.com>
@maliberty maliberty merged commit 0d236f4 into The-OpenROAD-Project:master Feb 27, 2025
11 checks passed
@maliberty maliberty deleted the secure-ave-cell-leakage branch February 27, 2025 23:03
@rovinski
Copy link
Collaborator

rovinski commented Mar 2, 2025

What's the point of taking the average state-based leakage? For all PDKs I can remember, the non-state-based leakage is already calculated as the average of the state-based leakages. This would only help if the average is weighted by state probability.

@precisionmoon
Copy link
Contributor

What's the point of taking the average state-based leakage? For all PDKs I can remember, the non-state-based leakage is already calculated as the average of the state-based leakages. This would only help if the average is weighted by state probability.

Correct, if a liberty has "cell_leakage_power" line, it is the average of the state-based leakages. Most public PDKs have the average leakage. However, many private PDK liberty files don't have this line. They only have multiple leakage values for different when conditions.

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.

5 participants