back.rtlil: do not squash empty modules.
authorwhitequark <whitequark@whitequark.org>
Wed, 26 Aug 2020 22:45:19 +0000 (22:45 +0000)
committerwhitequark <whitequark@whitequark.org>
Wed, 26 Aug 2020 22:45:19 +0000 (22:45 +0000)
commit07a3685da8e77199596dcaf7aedcc46fc602ce55
treed40b9c0fa3f9efe66d9e413e93029ee566a2173f
parent12beda6e5b1de4a3801098f14b819e91eb63e0f3
back.rtlil: do not squash empty modules.

In commit 9faa1d37, the RTLIL backend was changed to ignore modules
without ports completely, since Yosys would recognize empty modules
as black boxes without explicit `write_verilog -noblackbox` and break
the design. That change had many flaws:
  * It removed instances without ports, which are used in e.g. SoC
    FPGAs to instantiate a dummy CPU.
  * It removed fragments without ports, which can appear in e.g. SoC
    FPGAs in case the fabric is not connected to any I/O ports.
  * Finally, it was just conceptually unjustified.

This commit changes the logic to actually check for empty fragments,
and instead of removing them, it adds a dummy wire inside. It would
be possible to use the Yosys-specific (*noblackbox*) attribute.
However, it would be necessary to strip it for most targets right
away, and also the wire doubles as documentation.

Fixes #441.
nmigen/back/rtlil.py