driazati commented on code in PR #13040:
URL: https://github.com/apache/tvm/pull/13040#discussion_r992735082


##########
tests/scripts/task_config_build_hexagon.sh:
##########
@@ -28,7 +28,6 @@ echo set\(USE_RPC ON\) >> config.cmake
 echo set\(USE_MICRO ON\) >> config.cmake
 echo set\(USE_MICRO_STANDALONE_RUNTIME ON\) >> config.cmake
 echo set\(USE_LLVM "${CLANG_LLVM_HOME}/bin/llvm-config"\) >> config.cmake
-echo set\(CMAKE_CXX_COMPILER "/opt/sccache/clang++"\) >> config.cmake

Review Comment:
   It'd be fine to take this out, a bunch of people have had problems with it 
when running locally, but it should still stay on in CI. AFAIK all scripts that 
run in Jenkins during CI have the env variable `CI` set to `true`, could you 
switch on that and set `CMAKE_CXX_COMPILER` if that's the case?



-- 
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