[Feature](mlu-ops): remove error throw in mluops library. (#1038)

core/util.cpp

@@ -32,6 +32,9 @@ void mluOpCheck(mluOpStatus_t result, char const *const func,
     std::string error = "\"" + std::string(mluOpGetErrorString(result)) +
                         " in " + std::string(func) + "\"";
     LOG(ERROR) << error;
+    // TODO(liuduanhui): Remove error throwing in c library in future.
+    //                   MLUOP_CHECK should not be used in host side code.
+    //                   And now it is only used in gtest code.
     throw std::runtime_error(error);
   }
 }

kernels/utils/cnnl_helper.cpp

@@ -30,7 +30,6 @@ void mluOpCnnlCheck(mluOpStatus_t result, char const *const func,
         " in " + std::string(func) + " at " + std::string(file) +
         " line: " + std::to_string(line) + "\"";
     LOG(ERROR) << error;
-    throw std::runtime_error(error);
   }
 }
 

tools/pre-commit

@@ -68,8 +68,8 @@ for i in ${filenames} ; do
   check_dir=$(echo $i | grep -E "^core\/|^kernels\/")
   #not_in_tools=$(echo $i | sed -r "s/^tools.*//")
   if [[ -n "${include_mlu}" ]] && [[ -n "${check_dir}" ]]; then
-    printf_log=$(sed -n -e '/\<printf\>/=' -e ' \
-      /\<__bang_printf\>/=' -e '/\<std::cout\>/=' -e '/assert(/=' $i)
+    printf_log=$(sed -n -e '/\<printf\>/=' -e \
+      '/\<__bang_printf\>/=' -e '/\<std::cout\>/=' -e '/assert(/=' $i)
     for line in ${printf_log};
     do
       echo $i +${line}