Skip to content

Commit cf13512

Browse files
authored
Merge pull request #2952 from Pinata-Consulting/variables-add-flow-variant
variables: add FLOW_VARIANT
2 parents 95a6fe0 + 219b147 commit cf13512

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

docs/user/FlowVariables.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@ configuration file.
9494
| <a name="FILL_CELLS"></a>FILL_CELLS| Fill cells are used to fill empty sites. If not set or empty, fill cell insertion is skipped.| | |
9595
| <a name="FILL_CONFIG"></a>FILL_CONFIG| JSON rule file for metal fill during chip finishing.| | |
9696
| <a name="FLOORPLAN_DEF"></a>FLOORPLAN_DEF| Use the DEF file to initialize floorplan.| | |
97+
| <a name="FLOW_VARIANT"></a>FLOW_VARIANT| Flow variant to use, used in the flow variant directory name.| base| |
9798
| <a name="GDS_ALLOW_EMPTY"></a>GDS_ALLOW_EMPTY| Regular expression of module names of macros that have no .gds file| | |
9899
| <a name="GDS_FILES"></a>GDS_FILES| Path to platform GDS files.| | |
99100
| <a name="GENERATE_ARTIFACTS_ON_FAILURE"></a>GENERATE_ARTIFACTS_ON_FAILURE| For instance Bazel needs artifacts (.odb and .rpt files) on a failure to allow the user to save hours on re-running the failed step locally, but when working with a Makefile flow, it is more natural to fail the step and leave the user to manually inspect the logs and artifacts directly via the file system. Set to 1 to change the behavior to generate artifacts upon failure to e.g. do a global route. The exit code will still be non-zero on all other failures that aren't covered by the "useful to inspect the artifacts on failure" use-case. Example: just like detailed routing, a global route that fails with congestion, is not a build failure(as in exit code non-zero), it is a successful(as in zero exit code) global route that produce reports detailing the problem. Detailed route will not proceed, if there is global routing congestion This allows build systems, such as bazel, to create artifacts for global and detailed route, even if the operation had problems, without having know about the semantics between global and detailed route. Considering that global and detailed route can run for a long time and use a lot of memory, this allows inspecting results on a laptop for a build that ran on a server.| 0| |
@@ -392,6 +393,7 @@ configuration file.
392393
- [ENABLE_DPO](#ENABLE_DPO)
393394
- [FASTROUTE_TCL](#FASTROUTE_TCL)
394395
- [FILL_CONFIG](#FILL_CONFIG)
396+
- [FLOW_VARIANT](#FLOW_VARIANT)
395397
- [GDS_FILES](#GDS_FILES)
396398
- [GENERATE_ARTIFACTS_ON_FAILURE](#GENERATE_ARTIFACTS_ON_FAILURE)
397399
- [GLOBAL_PLACEMENT_ARGS](#GLOBAL_PLACEMENT_ARGS)

flow/Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,8 +162,6 @@ export SYNTH_MINIMUM_KEEP_SIZE ?= 0
162162
export SYNTH_OPERATIONS_ARGS ?= -extra-map $(FLOW_HOME)/platforms/common/lcu_kogge_stone.v
163163
export SYNTH_FULL_ARGS ?= $(SYNTH_ARGS) $(SYNTH_OPERATIONS_ARGS)
164164

165-
export FLOW_VARIANT ?= base
166-
167165
# Setup working directories
168166
export DESIGN_NICKNAME ?= $(DESIGN_NAME)
169167

flow/scripts/variables.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -915,3 +915,7 @@ YOSYS_FLAGS:
915915
Additional flags to pass to yosys.
916916
stages:
917917
- synth
918+
FLOW_VARIANT:
919+
description: >
920+
Flow variant to use, used in the flow variant directory name.
921+
default: base

0 commit comments

Comments
 (0)