You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/06/09 15:53:48 UTC

[GitHub] [tvm] areusch commented on pull request #8213: [LLVM] Fix CodeGenLLVM::LinkParameters

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:
users@infra.apache.org