Skip to content

Remove unhelpful axi wrapper runt tests and cocotb tests #2515

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 25, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 1 addition & 39 deletions runt.toml
Original file line number Diff line number Diff line change
Expand Up @@ -626,44 +626,6 @@ fud2 {} -s sim.cycle_limit=500 \
--to dat -q
"""

##### Calyx AXI tests #####
[[tests]]
name = "calyx-py read-write-compute AXI wrapper generation"
paths = ["yxi/tests/axi/read-compute-write/*.yxi"]
cmd = """
python3 yxi/axi-calyx/axi_generator.py {}
"""

[[tests]]
name = "calyx-py dynamic AXI wrapper generation"
paths = ["yxi/tests/axi/dynamic/*.yxi"]
cmd = """
python3 yxi/axi-calyx/dynamic_axi_generator.py {}
"""

#Ignore fud2 stderr for now due to ninja nondeterminism
[[tests]]
name = "fud2 AXI wrapper invocation"
paths = ["yxi/tests/axi/*/*-mem-vec-add.futil"]
cmd = """
dynamic=$(head -n 2 {} | tail -n 1 | cut -c 4-)
fud2 {} --to calyx --through axi-wrapped --set $dynamic 2> /dev/null
"""

[[tests]]
name = "fud2 AXI-wrapped to verilog"
paths = ["yxi/tests/axi/*/*-mem-vec-add-axi-wrapped.futil"]
cmd = """
fud2 {} --from calyx --to verilog-noverify 2> /dev/null
"""

[[tests]]
name = "fud2 cocotb AXI-wrapped xecution"
paths = ["yxi/tests/axi/*/*-mem-vec-add-verilog.v"]
cmd = """
fud2 {} --from verilog-noverify --to cocotb-axi --set sim.data=yxi/tests/axi/vectorized-add.data 2> /dev/null
"""

Comment on lines -660 to -666
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imo, this is the end goal test we want to eventually support because it tests end to end functionality.. Keeping everything up to date and having fud2 find a single path from .futil to .dat that goes through all the right steps (with multiple inputs) was the blocker on leaving this test without all the intermediate steps being tested for above it, iirc.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. Point taken: the thing we need to revive here is a way to test end-to-end execution, which currently requires multiple fud2 invocations (because of fundamental fud2 limitations).

##### Xilinx Tests ######
[[tests]]
name = "AXI wrapper generation"
Expand All @@ -683,7 +645,7 @@ target/debug/calyx {} -b xilinx-xml

[[tests]]
name = "kernel.xml generation from yxi"
paths = ["yxi/tests/axi/xml/*.yxi"]
paths = ["yxi/tests/axi/xml-gen/*.yxi"]
cmd = """
python3 yxi/xml/xml_generator.py {}
"""
Expand Down
Loading
Loading