Øyvind Harboe

Testing [CoralNPU](https://github.com/google-coral/coralnpu): bazelisk build //fpga:rtl_files Now do a quick build to see what we have... make SYNTH_HIERARCHICAL=1 SYNTH_MEMORY_MAX_BITS=1024 SYNTH_MINIMUM_KEEP_SIZE=0 SYNTH_HDL_FRONTEND=slang DESIGN_CONFIG=designs/asap7/minimal/config.mk DESIGN_NAME=SRAM_2048x128 VERILOG_FILES=~/coralnpu/bazel-bin/fpga/ip/coralnpu_chisel_subsystem_default/CoralNPUChiselSubsystem.sv do-yosys-canonicalize do-yosys This lists memories that are...

Yosys slang may be having trouble inferring memories. untar [yosys-no-memories.tar.gz](https://github.com/user-attachments/files/23442424/yosys-no-memories.tar.gz) Run: make --file=~/OpenROAD-flow-scripts/flow/Makefile SYNTH_HIERARCHICAL=0 SYNTH_MEMORY_MAX_BITS=1024 SYNTH_HDL_FRONTEND=slang DESIGN_CONFIG=~/OpenROAD-flow-scripts/flow/designs/asap7/minimal/config.mk DESIGN_NAME=Spi2TLUL SDC_CONSTRAINTS=~/OpenROAD-flow-scripts/flow/designs/asap7/mock-alu/constraints.sdc VERILOG_FILES=./home/oyvind/coralnpu/bazel-bin/fpga/ip/coralnpu_chisel_subsystem_default/CoralNPUChiselSubsystem.sv SDC_CONSTRAINTS=designs/asap7/mock-alu/constraints.sdc do-yosys-canonicalize do-yosys I expected this error to kick in:...

@povik FYI, unrelated to this PR as such, but when using `SYNTH_HDL_FRONTEND=slang`, yosys fails to infer memories. Untar [slang-no-infer-mem.tar.gz](https://github.com/user-attachments/files/23442480/slang-no-infer-mem.tar.gz) Run the below make --file=~/OpenROAD-flow-scripts/flow/Makefile SYNTH_HIERARCHICAL=0 SYNTH_MEMORY_MAX_BITS=1024 DESIGN_CONFIG=~/OpenROAD-flow-scripts/flow/designs/asap7/minimal/config.mk DESIGN_NAME=SRAM_2048x128 SDC_CONSTRAINTS=~/OpenROAD-flow-scripts/flow/designs/asap7/mock-alu/constraints.sdc VERILOG_FILES=blah.sv...

Recording notes while I kick off a larger run... ```patch diff --git a/flow/scripts/synth_preamble.tcl b/flow/scripts/synth_preamble.tcl index a37231275..c20bb2836 100644 --- a/flow/scripts/synth_preamble.tcl +++ b/flow/scripts/synth_preamble.tcl @@ -47,7 +47,7 @@ proc read_design_sources { } {...

This PR is good to go for my part now. Things are working pretty well. When I push yosys/synthesis hard enough, I, no surprise, find that there is always room...

add a link to officially published docker images? use git describe tag version in example, not hash

It is not a priority for me, but it still fails: ``` $ ./run-me-gcd-nangate45-base.sh OpenROAD v2.0-22248-ga7840fc2a6 Features included (+) or not (-): +GPU +GUI +Python This program is licensed under...

@jeffng-or FYI

@maliberty Add running times for now and close this issue? As you say, skew/CRPR is more of an in depth investigation than a simple metric issue.

> > Add running times for now and close this issue? > > Yes. Are you planning to make a PR or requesting PII do it? It is a feature...