makefile: move more variables setup to bash

Signed-off-by: Øyvind Harboe <[email protected]>

Author: oharboe

flow/Makefile

@@ -159,42 +159,6 @@ ifneq ($(BLOCKS),)
   endif
 endif
 
-#-------------------------------------------------------------------------------
-ifeq (,$(strip $(NUM_CORES)))
-  # Linux (utility program)
-  NUM_CORES := $(shell nproc 2>/dev/null)
-
-  ifeq (,$(strip $(NUM_CORES)))
-    # Linux (generic)
-    NUM_CORES := $(shell grep -c ^processor /proc/cpuinfo 2>/dev/null)
-  endif
-  ifeq (,$(strip $(NUM_CORES)))
-    # BSD (at least FreeBSD and Mac OSX)
-    NUM_CORES := $(shell sysctl -n hw.ncpu 2>/dev/null)
-  endif
-  ifeq (,$(strip $(NUM_CORES)))
-    # Fallback
-    NUM_CORES := 1
-  endif
-endif
-export NUM_CORES
-
-export YOSYS_FLAGS += -v 3
-
-# The following determine the executable location for each tool used by this flow.
-# Priority is given to
-#       1 user explicit set with variable in Makefile or command line, for instance setting OPENROAD_EXE
-#       2 ORFS compiled tools: openroad, yosys
-export OPENROAD_EXE      ?= $(abspath $(FLOW_HOME)/../tools/install/OpenROAD/bin/openroad)
-export OPENSTA_EXE       ?= $(abspath $(FLOW_HOME)/../tools/install/OpenROAD/bin/sta)
-
-export OPENROAD_ARGS            = -no_init -threads $(NUM_CORES) $(OR_ARGS)
-export OPENROAD_CMD             = $(OPENROAD_EXE) -exit $(OPENROAD_ARGS)
-export OPENROAD_NO_EXIT_CMD     = $(OPENROAD_EXE) $(OPENROAD_ARGS)
-export OPENROAD_GUI_CMD         = $(OPENROAD_EXE) -gui $(OR_ARGS)
-
-export YOSYS_EXE               ?= $(abspath $(FLOW_HOME)/../tools/install/yosys/bin/yosys)
-
 # Use locally installed and built klayout if it exists, otherwise use klayout in path
 KLAYOUT_DIR = $(abspath $(FLOW_HOME)/../tools/install/klayout/)
 KLAYOUT_BIN_FROM_DIR = $(KLAYOUT_DIR)/klayout

flow/scripts/config.sh

@@ -49,3 +49,46 @@ while IFS= read -r line; do
   # Replace "__SPACE__" with an actual space in the line
   eval "${line//__SPACE__/ }"
 done < <("${FLOW_HOME}/scripts/defaults.py")
+
+
+# Determine the number of cores (NUM_CORES)
+if [[ -z "${NUM_CORES:-}" ]]; then
+  # Linux (utility program)
+  NUM_CORES=$(nproc 2>/dev/null || true)
+
+  if [[ -z "$NUM_CORES" ]]; then
+    # Linux (generic)
+    NUM_CORES=$(grep -c ^processor /proc/cpuinfo 2>/dev/null || true)
+  fi
+
+  if [[ -z "$NUM_CORES" ]]; then
+    # BSD (at least FreeBSD and macOS)
+    NUM_CORES=$(sysctl -n hw.ncpu 2>/dev/null || true)
+  fi
+
+  if [[ -z "$NUM_CORES" ]]; then
+    # Fallback
+    NUM_CORES=1
+  fi
+fi
+export NUM_CORES
+
+# Determine the executable location for each tool used by this flow.
+# Priority is given to:
+# 1. User explicitly setting the variable in the environment or command line (e.g., OPENROAD_EXE).
+# 2. Default ORFS compiled tools: openroad, yosys.
+
+# Set default paths for executables if not already set
+export OPENROAD_EXE="${OPENROAD_EXE:-$(realpath "${FLOW_HOME}/../tools/install/OpenROAD/bin/openroad")}"
+export OPENSTA_EXE="${OPENSTA_EXE:-$(realpath "${FLOW_HOME}/../tools/install/OpenROAD/bin/sta")}"
+export YOSYS_EXE="${YOSYS_EXE:-$(realpath "${FLOW_HOME}/../tools/install/yosys/bin/yosys")}"
+
+export OR_ARGS="${OR_ARGS:-}"
+
+# Define OpenROAD command-line arguments
+export OPENROAD_ARGS="-no_init -threads ${NUM_CORES} ${OR_ARGS}"
+
+# Define OpenROAD commands
+export OPENROAD_CMD="${OPENROAD_EXE} -exit ${OPENROAD_ARGS}"
+export OPENROAD_NO_EXIT_CMD="${OPENROAD_EXE} ${OPENROAD_ARGS}"
+export OPENROAD_GUI_CMD="${OPENROAD_EXE} -gui ${OR_ARGS}"

flow/scripts/synth.sh

@@ -2,6 +2,8 @@
 set -u -eo pipefail
 mkdir -p $RESULTS_DIR $LOG_DIR $REPORTS_DIR $OBJECTS_DIR
 
+export YOSYS_FLAGS="${YOSYS_FLAGS:-} -v 3"
+
 # Not normally adjusted by user
 export SYNTH_OPERATIONS_ARGS="${SYNTH_OPERATIONS_ARGS:--extra-map ${FLOW_HOME}/platforms/common/lcu_kogge_stone.v}"
 export SYNTH_FULL_ARGS="${SYNTH_FULL_ARGS:=${SYNTH_ARGS} ${SYNTH_OPERATIONS_ARGS}}"