High-level structural comments, given that dpi part(verilog shell and c file) of the codebase is compiled with the verilator. We should consider a distinct folder structure. Here is one possible recommendation
``` /vta /vta/dpi - VTATSimMemHost.v - vta_tsim_dpi.h - vta_tsim_dpi.cc - README.md /vta/include/vta - dpi_module.h (the common dpi module defs that can be re-used) /vta/src - dpi_module.cc (consider links to vta instead of tvm runtime, import vta will import the dpi) ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/dmlc/tvm/pull/3010#issuecomment-483472352