efabless / openlane2

The next generation of OpenLane, rewritten from scratch with a modular architecture
https://openlane2.readthedocs.io/
Apache License 2.0
207 stars 38 forks source link

Fix arguments passed to Yosys hilomap #552

Closed htfab closed 1 month ago

htfab commented 1 month ago

In the Yosys synthesis script the hilomap command used for techmapping constant drivers is called with the wrong arguments. For instance, for sky130 it runs

hilomap -hicell sky130_fd_sc_hd__conb_1/HI -locell sky130_fd_sc_hd__conb_1/LO

instead of

hilomap -hicell sky130_fd_sc_hd__conb_1 HI -locell sky130_fd_sc_hd__conb_1 LO

This results in the step being skipped and some warnings:

56. Executing HILOMAP pass (mapping to constant drivers).
Warning: Selection "sky130_fd_sc_hd__conb_1" did not match any module.
Warning: Selection "LO" did not match any object.

This is a regression from OpenLane 1.