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

Reply via email to