Skip to content

Commit eebb6ec

Browse files
committed
back.verilog: require Yosys 0.38.
This avoids the awkward requirement due to the bug in Yosys 0.37, and will soon be required anyway once the `$check` cell is emitted.
1 parent 0da439c commit eebb6ec

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

amaranth/back/verilog.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@
99

1010
def _convert_rtlil_text(rtlil_text, *, strip_internal_attrs=False, write_verilog_opts=()):
1111
# this version requirement needs to be synchronized with the one in pyproject.toml!
12-
# Yosys 0.37 has a critical miscompilation in Verilog backend:
13-
# https://github.com/amaranth-lang/amaranth/issues/1049
14-
yosys = find_yosys(lambda ver: ver >= (0, 35) and not (0, 36, 79) <= ver <= (0, 37, 29))
12+
yosys = find_yosys(lambda ver: ver >= (0, 38))
1513

1614
script = []
1715
script.append(f"read_ilang <<rtlil\n{rtlil_text}\nrtlil")

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ dependencies = [
2020

2121
[project.optional-dependencies]
2222
# this version requirement needs to be synchronized with the one in amaranth.back.verilog!
23-
builtin-yosys = ["amaranth-yosys>=0.35,!=0.37.0.*"]
23+
builtin-yosys = ["amaranth-yosys>=0.38"]
2424
remote-build = ["paramiko~=2.7"]
2525

2626
[project.scripts]

0 commit comments

Comments
 (0)