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). Here are how we interpret values to keep things consistent with our previous options: - 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. The use of ccache will likely cause the cache size growing problem in the CI. because the cost of build is normally as large compared to testing, we will likely want to disable ccache in the CI setups in the task_config_build_xyz.sh https://github.com/apache/tvm/blob/main/tests/scripts/task_config_build_arm.sh -- 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]
