aasorokiin commented on a change in pull request #8797:
URL: https://github.com/apache/tvm/pull/8797#discussion_r695357158
##########
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:
Here is what worked for me. I removed verilated.cpp and
verilated_dpi.cpp from cmake config src object list and included them in vta
module.cc itself in 3rdparty/vta-hardware. Make compiles those sv functions
right into module.cc object file where they are used. Somehow it doesn't flag
warnings on compilation, but those needed functions got created. We will not
see if it is the case till updated VTA is connected to this TVM. In this TVM
those cpp files are not compiled and not required anywhere.
Disabling redefinition warning did not work as I was not able to find -W
option for it. @stevenmburns
--
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]
