[GitHub] [tvm] vegaluisjose commented on a change in pull request #8797: VTA cmake change to include Verilator header for building tsim library

[GitBox]

vegaluisjose commented on a change in pull request #8797:
URL: https://github.com/apache/tvm/pull/8797#discussion_r694316957



##########
File path: cmake/modules/VTA.cmake
##########
@@ -73,15 +73,30 @@ elseif(PYTHON)
 
   # Cycle accurate simulator driver build
   if(USE_VTA_TSIM)
+    if(NOT DEFINED ENV{VERILATOR_INC_DIR})

Review comment:
       (1) I am not sure, CI has been in flux recently.
   (2) I don't think it is possible but do you know if newer version of 
[Verilator](https://github.com/apache/tvm/blob/main/docker/install/ubuntu_install_verilator.sh#L24)
 might help?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tvm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org