usb-tools/amaranth
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

back.pysim: emit toplevel inputs in VCD files as well.

Browse source 
Before this commit, only signals driven from fragments (in practice,
everything except toplevel inputs) would get written to a VCD file.
Not having toplevel inputs in the dump made debugging ~impossible.

After this commit, all signals the fragment refers to get written to
a VCD file. (More specifically, all signals the compiler assigns
an index to, i.e. signals the generated code reads or writes.)

Fixes #280.
This commit is contained in:
whitequark 2020-02-06 17:19:47 +00:00
parent d3775eedd7
commit 882fddfa96
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
nmigen/back/pysim.py
View file

@ -704,9 +704,6 @@ class _FragmentCompiler:
self.signal_names[signal].add(hierarchical_signal_name)
for domain_name, domain_signals in fragment.drivers.items():
for domain_signal in domain_signals:
add_signal_name(domain_signal)
domain_stmts = LHSGroupFilter(domain_signals)(fragment.statements)
domain_process = _CompiledProcess(self.state, comb=domain_name is None,
name=".".join((*hierarchy, "<{}>".format(domain_name or "comb"))))
@ -762,6 +759,9 @@ class _FragmentCompiler:
processes.add(domain_process)
for used_signal in domain_process.context.indexes:
add_signal_name(used_signal)
for subfragment_index, (subfragment, subfragment_name) in enumerate(fragment.subfragments):
if subfragment_name is None:
subfragment_name = "U${}".format(subfragment_index)