Merge pull request #2537 from Pinata-Consulting/synth-stats-naming

synthesis: simpler more consistent naming for gathering statistics

Author: maliberty

flow/Makefile

@@ -181,7 +181,7 @@ include $(PLATFORM_DIR)/config.mk
 # Enables hierarchical yosys
 export SYNTH_HIERARCHICAL ?= 0
 export SYNTH_STATS = $(RESULTS_DIR)/synth_stats.txt
-export HIER_REPORT_SCRIPT = $(SCRIPTS_DIR)/synth_hier_report.tcl
+export SYNTH_STATS_SCRIPT = $(SCRIPTS_DIR)/synth_stats.tcl
 export MAX_UNGROUP_SIZE ?= 0
 
 # Enables Re-synthesis for area reclaim
@@ -475,11 +475,11 @@ memory:
 export SYNTH_SCRIPT ?= $(SCRIPTS_DIR)/synth.tcl
 export SYNTH_MEMORY_MAX_BITS ?= 4096
 
-.PHONY: do-yosys-keep-hierarchy
-do-yosys-keep-hierarchy:
+.PHONY: do-yosys-stats
+do-yosys-stats:
 	mkdir -p $(RESULTS_DIR) $(LOG_DIR) $(REPORTS_DIR)
 	(export VERILOG_FILES=$(RESULTS_DIR)/1_synth.rtlil; \
-	$(TIME_CMD) $(YOSYS_EXE) $(YOSYS_FLAGS) -c $(HIER_REPORT_SCRIPT)) 2>&1 | tee $(abspath $(LOG_DIR)/1_1_yosys_hier_report.log)
+	$(TIME_CMD) $(YOSYS_EXE) $(YOSYS_FLAGS) -c $(SYNTH_STATS_SCRIPT)) 2>&1 | tee $(abspath $(LOG_DIR)/1_1_yosys_stats.log)
 
 export SDC_FILE_CLOCK_PERIOD = $(RESULTS_DIR)/clock_period.txt
 
@@ -507,7 +507,7 @@ $(RESULTS_DIR)/1_synth.rtlil: $(YOSYS_DEPENDENCIES)
 	$(UNSET_AND_MAKE) do-yosys-canonicalize
 
 $(RESULTS_DIR)/1_1_yosys.v: $(RESULTS_DIR)/1_synth.rtlil
-	$(UNSET_AND_MAKE) do-yosys-keep-hierarchy do-yosys
+	$(UNSET_AND_MAKE) do-yosys-stats do-yosys
 
 .PHONY: do-synth
 do-synth:

flow/scripts/synth_hier_report.tcl flow/scripts/synth_stats.tclflow/scripts/synth_hier_report.tcl renamed to flow/scripts/synth_stats.tcl

@@ -1,3 +1,8 @@
+# Gather statistics, if needed. Gathering the statistics is
+# time consuming, because it requires a full synthesis run.
+#
+# The statistics is needed for SYNTH_HIERARCHICAL=1 flattening/keep
+# module policy, and write to $::env(SYNTH_STATS)
 source $::env(SCRIPTS_DIR)/util.tcl
 
 proc write_keep_hierarchy {} {
@@ -51,8 +56,8 @@ proc write_keep_hierarchy {} {
 
   set areas {}
   foreach module $module_list {
-    tee -o $::env(OBJECTS_DIR)/synth_hier_stat_temp_module.txt stat -top "$module" {*}$stat_libs
-    set fptr1 [open $::env(OBJECTS_DIR)/synth_hier_stat_temp_module.txt r]
+    tee -o $::env(OBJECTS_DIR)/synth_stat_temp_module.txt stat -top "$module" {*}$stat_libs
+    set fptr1 [open $::env(OBJECTS_DIR)/synth_stat_temp_module.txt r]
     set contents1 [read -nonewline $fptr1]
     close $fptr1
     set split_cont1 [split $contents1 "\n"]
@@ -61,7 +66,7 @@ proc write_keep_hierarchy {} {
         lappend areas "$area $module_name"
       }
     }
-    file delete -force $::env(OBJECTS_DIR)/synth_hier_stat_temp_module.txt
+    file delete -force $::env(OBJECTS_DIR)/synth_stat_temp_module.txt
   }
   set areas [lsort -index 0 -real $areas]