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/30 12:59:23 UTC

[GitHub] [tvm] tqchen edited a comment on pull request #8373: Add caching to CMake

tqchen edited a comment on pull request #8373:
URL: https://github.com/apache/tvm/pull/8373#issuecomment-871382068


   Thanks @electriclilies . One thing that might be helpful here is to streamline the option definition. The current proposed option is closer to `AUTO` mode(enable if found, but disable if not).
   
   The general possible usages here:
   - AUTO: automatically detect and if found use it
   - /path/to/ccache: set the ccache to the correct ccache path being specified
   - OFF: disable ccache
   - ON: search and report an error if ccache is not found
   
   We should also look a bit into which way is more standard, since I also see `set(CMAKE_C_COMPILER_LAUNCHER /path/to/ccache)` being used a bit more frequently.
   
   
   
   
   


-- 
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: commits-unsubscribe@tvm.apache.org

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