From dc9ee6d63fc15a12e1337860117344a6d9f44505 Mon Sep 17 00:00:00 2001 From: "Steven S. Lyubomirsky" Date: Mon, 6 Dec 2021 21:43:31 -0800 Subject: [PATCH] Remove things that use our VTA fork and will not work now --- README.md | 8 -------- vta/python/vta/testing/simulator.py | 9 ++++++--- vta/tests/python/integration/matmul_tutorial.py | 3 --- 3 files changed, 6 insertions(+), 14 deletions(-) diff --git a/README.md b/README.md index 969b89c5e..2991b1a20 100644 --- a/README.md +++ b/README.md @@ -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 diff --git a/vta/python/vta/testing/simulator.py b/vta/python/vta/testing/simulator.py index cd7c55aa7..43fcafc6c 100644 --- a/vta/python/vta/testing/simulator.py +++ b/vta/python/vta/testing/simulator.py @@ -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): @@ -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): @@ -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() diff --git a/vta/tests/python/integration/matmul_tutorial.py b/vta/tests/python/integration/matmul_tutorial.py index a9c020c65..f6917cd8d 100644 --- a/vta/tests/python/integration/matmul_tutorial.py +++ b/vta/tests/python/integration/matmul_tutorial.py @@ -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