Skip to content

Commit

Permalink
Remove things that use our VTA fork and will not work now
Browse files Browse the repository at this point in the history
  • Loading branch information
slyubomirsky authored and gussmith23 committed Dec 15, 2021
1 parent c389c23 commit dc9ee6d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 14 deletions.
8 changes: 0 additions & 8 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,14 +19,6 @@ This is a fork of TVM for adding BYOC integrations for the 3LA project.

Right now we have a VTA integration in `src/relay/backend/contrib/ilavta`. Note that you have to include the line `SET(USE_ILAVTA_CODEGEN ON)` in `build/config.cmake` before building TVM to support this (other flags that should be on: `USE_LLVM`, `USE_VTA_FSIM`). We have a test of this backend in `tests/python/relay/test_external_codegen.py` (see `test_extern_vta()`).

This version also uses a fork of the VTA repo meant to dump logs.
Try `vta/python/integration/matmul_tutorial.py` to use the dumping facility.
VTA can be set into dumping mode by calling `vta.testing.simulator.dump_mode(True)`.
You can specify the location at which the dump will be deposited using `vta.testing.simulator.dump_target(path)`; the default is `./vta_sim_dump.json`.
See the readme at [the VTA fork](https://github.com/uwsampl/3la-vta) to see a description of the dumping mode and the dumping format.

You can use `vta.testing.ila_converter.convert(dump_file, dest_file)` to convert a VTA simulator dump into an ILA program fragment.

# 3LA environment setup

## Docker setup
Expand Down
9 changes: 6 additions & 3 deletions vta/python/vta/testing/simulator.py
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,8 @@ def dump_mode(toggle):
Whether to dump logs and memory
(false by default)
"""
tvm.get_global_func("vta.simulator.profiler_dump_mode")(1 if toggle else 0)
return
# tvm.get_global_func("vta.simulator.profiler_dump_mode")(1 if toggle else 0)


def sim_dump_target(target):
Expand All @@ -129,7 +130,8 @@ def sim_dump_target(target):
and memory dump corresponding to the initial state
if dumping mode is on.
"""
tvm.get_global_func("vta.simulator.profiler_dump_target")(target)
return
# tvm.get_global_func("vta.simulator.profiler_dump_target")(target)


def output_dump_target(target):
Expand All @@ -141,6 +143,7 @@ def output_dump_target(target):
and memory dump corresponding to
all stores to DRAM if dumping mode is on.
"""
tvm.get_global_func("vta.simulator.profiler_output_dump_target")(target)
return
# tvm.get_global_func("vta.simulator.profiler_output_dump_target")(target)

LIBS = _load_sw()
3 changes: 0 additions & 3 deletions vta/tests/python/integration/matmul_tutorial.py
Original file line number Diff line number Diff line change
Expand Up @@ -432,9 +432,6 @@

# Clear stats
if env.TARGET in ["sim", "tsim"]:
simulator.dump_mode(True)
simulator.sim_dump_target("test_sim.json")
simulator.output_dump_target("test_output.json")
simulator.clear_stats()

# Invoke the module to perform the computation
Expand Down

0 comments on commit dc9ee6d

Please sign in to comment.