areusch commented on pull request #8213:
URL: https://github.com/apache/tvm/pull/8213#issuecomment-857824746


   @kparzysz-quic yeah i think you're right--we get LLVM from APT in the docker 
container, which i doubt has `-DLLVM_ENABLE_ASSERTIONS` as the website mentions 
it's slow. For that reason, I'm hesitant to suggest changing our LLVM install 
process to build one with assertions as it might significantly impact the CI 
speed. we could potentially add one version which contains assertions and run 
codegen unit tests against that. what are your thoughts?


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

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


Reply via email to