* review comments

src/relay/backend/contrib/clml/codegen.cc

@@ -328,11 +328,7 @@ class CLMLJSONSerializer : public backend::contrib::JSONSerializer {
     const auto* dense = fn->body.as<CallNode>();
     const CallNode* bias = nullptr;
 
-    if (backend::IsOp(dense, "add")) {
-      bias = dense;
-      dense = dense->args[0].as<CallNode>();
-    }
-    if (backend::IsOp(dense, "nn.bias_add")) {
+    if (backend::IsOp(dense, "add") || backend::IsOp(dense, "nn.bias_add")) {
       bias = dense;
       dense = dense->args[0].as<CallNode>();
     }

src/runtime/contrib/clml/clml_runtime.cc

@@ -170,7 +170,8 @@ class CLMLRuntime : public JSONRuntimeBase {
     }
     ICHECK(h_ClmlIntf != NULL)
         << "clGetMLInterfaceVxQCOM:" << result
-        << " Perhaps there is mispatch between CLML SDK version to target supported version";
+        << " Perhaps there is mispatch between CLML SDK version to target supported version:"
+        << majorVersions[numVersions - 1];
     char* tune_flag;
     if ((tune_flag = getenv("CLML_IS_TUNNING_RUN")))
       this->is_tuning_run = std::stoi(tune_flag);

tests/python/contrib/test_clml/test_ops.py

@@ -120,7 +120,6 @@ def _get_conv_expected_codegen(
         weight_shape = (channels, shape[1] // groups, kernel_h, kernel_w)
     else:
         weight_shape = (shape[1] // groups, channels, kernel_h, kernel_w)
-    # weight_shape = (channels, shape[1] // groups, kernel_h, kernel_w)
 
     if is_depthwise:
         name = "nn.depthwise_conv2d"

tests/scripts/task_config_build_adreno.sh

@@ -23,8 +23,6 @@ mkdir -p "$BUILD_DIR"
 cd "$BUILD_DIR"
 cp ../cmake/config.cmake .
 
-[[ -z "${ADRENO_OPENCL}" ]] && CLML_PATH='ON' || CLML_PATH="${ADRENO_OPENCL}"
-
 echo set\(USE_OPENCL ON\) >> config.cmake
 echo set\(USE_CLML ${CLML_PATH}\) >> config.cmake
 echo set\(USE_RPC ON\) >> config.cmake