Skip to content

updated cluster_flops enablement check #4143

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

Merged
maliberty merged 1 commit into from
Apr 14, 2026
+13 −4
Merged

updated cluster_flops enablement check #4143

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion docs/user/FlowVariables.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,8 @@ configuration file.
| <a name="CELL_PAD_IN_SITES_DETAIL_PLACEMENT"></a>CELL_PAD_IN_SITES_DETAIL_PLACEMENT| Cell padding on both sides in site widths to ease routability in detail placement.| 0|
| <a name="CELL_PAD_IN_SITES_GLOBAL_PLACEMENT"></a>CELL_PAD_IN_SITES_GLOBAL_PLACEMENT| Cell padding on both sides in site widths to ease routability during global placement.| 0|
| <a name="CLKGATE_MAP_FILE"></a>CLKGATE_MAP_FILE| Optional mapping file supplied to Yosys to map clock gating cells| |
| <a name="CLUSTER_FLOPS"></a>CLUSTER_FLOPS| Minimum number of flip-flops per sink cluster.| 0|
| <a name="CLUSTER_FLOPS"></a>CLUSTER_FLOPS| Enable clustering of flip-flops into multi-bit flip-flops, if the platform PDK includes multi-bit flip-flops| 0|
| <a name="CLUSTER_FLOPS_ARGS"></a>CLUSTER_FLOPS_ARGS| Additional arguments passed to the cluster_flops command.| |
Comment on lines +112 to +113
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This file is automatically generated from flow/scripts/variables.yaml (as noted on line 87). Manual edits to these tables and lists are likely to be overwritten. It is recommended to update the documentation by running the generation script after modifying variables.yaml to ensure consistency and avoid maintenance issues.

| <a name="CORE_AREA"></a>CORE_AREA| The core area specified as a list of lower-left and upper-right corners in microns (X1 Y1 X2 Y2).| |
| <a name="CORE_ASPECT_RATIO"></a>CORE_ASPECT_RATIO| The core aspect ratio (height / width). This variable is only used when `CORE_UTILIZATION` is set.| 1.0|
| <a name="CORE_MARGIN"></a>CORE_MARGIN| The margin between the core area and die area, specified in microns. Allowed values are either one value for all margins or a set of four values, one for each margin. The order of the four values are: `{bottom top left right}`. This variable is only used when `CORE_UTILIZATION` is set.| 1.0|
Expand Down Expand Up @@ -434,6 +435,7 @@ configuration file.
- [CELL_PAD_IN_SITES_DETAIL_PLACEMENT](#CELL_PAD_IN_SITES_DETAIL_PLACEMENT)
- [CELL_PAD_IN_SITES_GLOBAL_PLACEMENT](#CELL_PAD_IN_SITES_GLOBAL_PLACEMENT)
- [CLUSTER_FLOPS](#CLUSTER_FLOPS)
- [CLUSTER_FLOPS_ARGS](#CLUSTER_FLOPS_ARGS)
- [DETAIL_PLACEMENT_ARGS](#DETAIL_PLACEMENT_ARGS)
- [DONT_BUFFER_PORTS](#DONT_BUFFER_PORTS)
- [EARLY_SIZING_CAP_RATIO](#EARLY_SIZING_CAP_RATIO)
Expand Down
4 changes: 2 additions & 2 deletions flow/scripts/global_place.tcl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ if { $result != 0 } {

log_cmd estimate_parasitics -placement

if { [env_var_exists_and_non_empty CLUSTER_FLOPS] } {
cluster_flops {*}[env_var_or_empty CLUSTER_FLOPS_ARGS]
if { $::env(CLUSTER_FLOPS) } {
log_cmd cluster_flops {*}[env_var_or_empty CLUSTER_FLOPS_ARGS]
log_cmd estimate_parasitics -placement
}

Expand Down
9 changes: 8 additions & 1 deletion flow/scripts/variables.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1001,10 +1001,17 @@ CTS_BUF_LIST:
- cts
CLUSTER_FLOPS:
description: >
Minimum number of flip-flops per sink cluster.
Enable clustering of flip-flops into multi-bit flip-flops, if the platform
PDK includes multi-bit flip-flops
stages:
- place
default: 0
CLUSTER_FLOPS_ARGS:
description: >
Additional arguments passed to the cluster_flops command.
stages:
- place
default: ""
CTS_CLUSTER_DIAMETER:
description: >
Maximum diameter (in microns) of sink cluster.
Expand Down