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



##########
File path: tests/scripts/task_config_build_cpu.sh
##########
@@ -34,7 +34,6 @@ echo set\(USE_NNPACK ON\) >> config.cmake
 echo set\(NNPACK_PATH /NNPACK/build/\) >> config.cmake
 echo set\(USE_ANTLR ON\) >> config.cmake
 echo set\(CMAKE_CXX_COMPILER g++\) >> config.cmake
-echo set\(CMAKE_CXX_FLAGS -Werror\) >> config.cmake

Review comment:
       -Werror is present in all other configurations which don't build with 
Verilator. Pulled back it will cause lib_tsim.so compilation failure due to 
warnings in Verilator source code.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to