srkreddy1238 commented on code in PR #13670:
URL: https://github.com/apache/tvm/pull/13670#discussion_r1059568806
##########
cmake/modules/contrib/CLML.cmake:
##########
@@ -22,7 +22,21 @@ if(USE_CLML)
if(NOT USE_CLML_GRAPH_EXECUTOR)
list(APPEND COMPILER_SRCS ${CLML_RUNTIME_MODULE})
endif()
- message(STATUS "Build with CLML support...")
+ message(STATUS "Build with CLML support : " ${USE_CLML})
+ if (NOT USE_CLML STREQUAL "ON")
+ set(CLML_VERSION_HEADER "${USE_CLML}/CL/cl_qcom_ml_ops.h")
+ if(EXISTS ${CLML_VERSION_HEADER})
+ file(READ ${CLML_VERSION_HEADER} ver)
+ string(REGEX MATCH "CL_QCOM_ML_OPS_H_MAJOR_VERSION ([0-9]*)" _
${ver})
+ set(CLML_VERSION_MAJOR ${CMAKE_MATCH_1})
Review Comment:
For codegen, OpenCL1.2 restriction is good as we don't have any requirement
of using features from higher versions.
CLML has newer versions with new operators support and is essential to
support to get perf improvements on new gerations.
--
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]
