diff --git a/.github/workflows/code_style.yml b/.github/workflows/code_style.yml index 8f69ee7cd4ca74..89527187272227 100644 --- a/.github/workflows/code_style.yml +++ b/.github/workflows/code_style.yml @@ -39,7 +39,11 @@ jobs: with: github_token: ${{ secrets.GITHUB_TOKEN }} level: warning - fail_on_error: true + fail_level: error + filter_mode: nofilter + exclude: | + "*/thirdparty/*" + "./temp/*" clang-format-aarch64: runs-on: ubuntu-22.04 @@ -71,7 +75,11 @@ jobs: with: github_token: ${{ secrets.GITHUB_TOKEN }} level: warning - fail_on_error: true + fail_level: error + filter_mode: nofilter + exclude: | + "*/thirdparty/*" + "./temp/*" ShellCheck: runs-on: ubuntu-22.04 @@ -103,7 +111,7 @@ jobs: level: style reporter: github-pr-review check_all_files_with_shebangs: true - fail_on_error: true + fail_level: error exclude: | "*/thirdparty/*" "./temp/*" diff --git a/src/frontends/onnx/frontend/src/utils/onnx_internal.cpp b/src/frontends/onnx/frontend/src/utils/onnx_internal.cpp index a5b68b8f9d98a5..96aad10dbf928c 100644 --- a/src/frontends/onnx/frontend/src/utils/onnx_internal.cpp +++ b/src/frontends/onnx/frontend/src/utils/onnx_internal.cpp @@ -100,7 +100,8 @@ std::shared_ptr decode_to_framework_nodes(std::shared_ptr detail::MappedMemoryHandles mmap_cache, ov::frontend::ExtensionHolder extensions) { apply_transformations(*model_proto); - auto graph = std::make_shared(ov::util::get_directory(model_path).string(), model_proto, mmap_cache, extensions); + auto graph = + std::make_shared(ov::util::get_directory(model_path).string(), model_proto, mmap_cache, extensions); return graph->decode(); } } // namespace ov::frontend::onnx::detail diff --git a/src/frontends/onnx/tests/onnx_import_convpool.in.cpp b/src/frontends/onnx/tests/onnx_import_convpool.in.cpp index cd944d44929abf..ae9188b8e7df5e 100644 --- a/src/frontends/onnx/tests/onnx_import_convpool.in.cpp +++ b/src/frontends/onnx/tests/onnx_import_convpool.in.cpp @@ -426,14 +426,17 @@ OPENVINO_TEST(${BACKEND_NAME}, onnx_model_convtranspose_output_shape) { test_case.add_input_from_file(util::path_join({ov::test::utils::getExecutableDirectory(), TEST_ONNX_MODELS_DIRNAME, - "files/convtranspose_output_shape/x.bin"}).string()); + "files/convtranspose_output_shape/x.bin"}) + .string()); test_case.add_input_from_file(util::path_join({ov::test::utils::getExecutableDirectory(), TEST_ONNX_MODELS_DIRNAME, - "files/convtranspose_output_shape/w.bin"}).string()); + "files/convtranspose_output_shape/w.bin"}) + .string()); test_case.add_expected_output_from_file({1, 2, 10, 8}, util::path_join({ov::test::utils::getExecutableDirectory(), TEST_ONNX_MODELS_DIRNAME, - "files/convtranspose_output_shape/y.bin"}).string()); + "files/convtranspose_output_shape/y.bin"}) + .string()); test_case.run(); } diff --git a/src/frontends/tensorflow/src/graph_iterator_saved_model.cpp b/src/frontends/tensorflow/src/graph_iterator_saved_model.cpp index 91ac2efc73d834..8d64d6f2bcb045 100644 --- a/src/frontends/tensorflow/src/graph_iterator_saved_model.cpp +++ b/src/frontends/tensorflow/src/graph_iterator_saved_model.cpp @@ -43,8 +43,7 @@ bool GraphIteratorSavedModel::is_supported(const std::string& path) { #if defined(OPENVINO_ENABLE_UNICODE_PATH_SUPPORT) && defined(_WIN32) bool GraphIteratorSavedModel::is_supported(const std::wstring& path) { - return ov::util::directory_exists(path) && - ov::util::file_exists(ov::util::path_join_w({path, L"saved_model.pb"})); + return ov::util::directory_exists(path) && ov::util::file_exists(ov::util::path_join_w({path, L"saved_model.pb"})); } #endif diff --git a/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.cpp b/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.cpp index bac1591dc0b76a..3fbdc645a03a4f 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.cpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.cpp @@ -124,9 +124,9 @@ void jit_loop_end_emitter::validate_arguments(const std::vector& in, con } void jit_loop_end_emitter::emit_code_impl(const std::vector& in, - const std::vector& out, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const { + const std::vector& out, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const { validate_arguments(in, out); emit_impl(in, out); } diff --git a/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.hpp b/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.hpp index eddfe64cdc90dd..8f3480cc064c7b 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.hpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/aarch64/jit_loop_emitters.hpp @@ -54,9 +54,9 @@ class jit_loop_end_emitter : public jit_emitter { } void emit_code_impl(const std::vector& in_idxs, - const std::vector& out_idxs, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const override; + const std::vector& out_idxs, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const override; protected: void validate_arguments(const std::vector& in, const std::vector& out) const override; diff --git a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.cpp b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.cpp index 15c7b85c1928da..54597a90a8504b 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.cpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.cpp @@ -200,9 +200,9 @@ void jit_loop_end_emitter::validate_arguments(const std::vector& in, con } void jit_loop_end_emitter::emit_code_impl(const std::vector& in, - const std::vector& out, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const { + const std::vector& out, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const { validate_arguments(in, out); jit_emitter::emit_code_impl(in, out, pool_vec_idxs, pool_gpr_idxs); } diff --git a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.hpp b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.hpp index 7b66cd8b54b48a..3a15ab0cf6fbea 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.hpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_loop_emitters.hpp @@ -67,9 +67,9 @@ class jit_loop_end_emitter : public jit_emitter { } void emit_code_impl(const std::vector& in_idxs, - const std::vector& out_idxs, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const override; + const std::vector& out_idxs, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const override; protected: void validate_arguments(const std::vector& in, const std::vector& out) const override; diff --git a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.cpp b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.cpp index 59f84c75ee28ee..8c848567f579e0 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.cpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.cpp @@ -71,9 +71,9 @@ void jit_reg_spill_end_emitter::validate_arguments(const std::vector& in } void jit_reg_spill_end_emitter::emit_code_impl(const std::vector& in, - const std::vector& out, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const { + const std::vector& out, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const { validate_arguments(in, out); emit_impl(in, out); } diff --git a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.hpp b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.hpp index a67caeb15ed6fb..0c7518d9df4b07 100644 --- a/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.hpp +++ b/src/plugins/intel_cpu/src/emitters/snippets/x64/jit_reg_spill_emitters.hpp @@ -27,9 +27,9 @@ class jit_reg_spill_begin_emitter : public jit_emitter { void validate_arguments(const std::vector& in, const std::vector& out) const override; void emit_impl(const std::vector& in, const std::vector& out) const override; void emit_code_impl(const std::vector& in_idxs, - const std::vector& out_idxs, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const override; + const std::vector& out_idxs, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const override; std::set m_regs_to_spill; std::shared_ptr m_abi_reg_spiller; }; @@ -49,9 +49,9 @@ class jit_reg_spill_end_emitter : public jit_emitter { } void emit_code_impl(const std::vector& in_idxs, - const std::vector& out_idxs, - const std::vector& pool_vec_idxs, - const std::vector& pool_gpr_idxs) const override; + const std::vector& out_idxs, + const std::vector& pool_vec_idxs, + const std::vector& pool_gpr_idxs) const override; protected: void validate_arguments(const std::vector& in, const std::vector& out) const override; diff --git a/src/plugins/intel_cpu/src/nodes/executors/aarch64/jit_eltwise.cpp b/src/plugins/intel_cpu/src/nodes/executors/aarch64/jit_eltwise.cpp index d2238b2d9f182c..492bf9d2899790 100644 --- a/src/plugins/intel_cpu/src/nodes/executors/aarch64/jit_eltwise.cpp +++ b/src/plugins/intel_cpu/src/nodes/executors/aarch64/jit_eltwise.cpp @@ -47,7 +47,7 @@ bool JitEltwiseExecutor::isSupported(const Algorithm& algorithm, Algorithm::EltwiseMod, Algorithm::EltwiseMultiply, Algorithm::EltwiseMulAdd, - Algorithm::EltwiseNotEqual, + Algorithm::EltwiseNotEqual, Algorithm::EltwisePowerStatic, Algorithm::EltwisePrelu, Algorithm::EltwiseRelu, diff --git a/src/plugins/intel_cpu/src/nodes/kernels/aarch64/jit_uni_eltwise_generic.cpp b/src/plugins/intel_cpu/src/nodes/kernels/aarch64/jit_uni_eltwise_generic.cpp index 818d7bdbfef684..ca82079b58ba68 100644 --- a/src/plugins/intel_cpu/src/nodes/kernels/aarch64/jit_uni_eltwise_generic.cpp +++ b/src/plugins/intel_cpu/src/nodes/kernels/aarch64/jit_uni_eltwise_generic.cpp @@ -29,7 +29,10 @@ void jit_uni_eltwise_generic::generate() { preamble(); static const std::vector exec_precisions_priority = {element::f16, element::f32}; - auto const exec_prc = eltwise_precision_helper::get_precision(jep_.inputs_number, jep_.src_prc, eltwise_data_, exec_precisions_priority); + auto const exec_prc = eltwise_precision_helper::get_precision(jep_.inputs_number, + jep_.src_prc, + eltwise_data_, + exec_precisions_priority); eltwise_emitter = create_eltwise_emitter(eltwise_data_.front(), exec_prc); for (size_t i = 1; i < eltwise_data_.size(); ++i) { @@ -46,8 +49,7 @@ void jit_uni_eltwise_generic::generate() { for (size_t i = 0; i < jep.inputs_number; i++) { ldr(start_to_offsets, ptr(reg_const_params, - static_cast(offsetof(jit_eltwise_call_args_ptrs, src_offsets) + - i * sizeof(size_t)))); + static_cast(offsetof(jit_eltwise_call_args_ptrs, src_offsets) + i * sizeof(size_t)))); ldr(get_src_reg(i), ptr(reg_const_params, static_cast(offsetof(jit_eltwise_call_args_ptrs, src_ptr[0]) + i * sizeof(size_t)))); @@ -91,8 +93,7 @@ void jit_uni_eltwise_generic::generate() { for (size_t i = 0; i < jep.inputs_number; i++) { ldr(get_src_reg(i), - ptr(param1, - static_cast(offsetof(jit_eltwise_call_args_ptrs, src_ptr) + i * sizeof(size_t)))); + ptr(param1, static_cast(offsetof(jit_eltwise_call_args_ptrs, src_ptr) + i * sizeof(size_t)))); init_ptrs_with_offsets(get_src_reg(i), jep.src_offsets[i]); } @@ -786,7 +787,6 @@ struct SupportedPrecisions { }; } // namespace - using namespace aarch64; std::set> eltwise_precision_helper::get_supported_precisions(const Algorithm& algo) { diff --git a/src/plugins/intel_cpu/src/nodes/kernels/x64/mlp_kernel.hpp b/src/plugins/intel_cpu/src/nodes/kernels/x64/mlp_kernel.hpp index 438d84c16b3ece..e3aae6ab38eaec 100644 --- a/src/plugins/intel_cpu/src/nodes/kernels/x64/mlp_kernel.hpp +++ b/src/plugins/intel_cpu/src/nodes/kernels/x64/mlp_kernel.hpp @@ -529,8 +529,8 @@ class ReduceAdd2bh : public dnnl::impl::cpu::x64::jit_generator { struct CallArgs { float* src0; float* src1; - int16_t * dst; - int16_t * prefetch_dst; + int16_t* dst; + int16_t* prefetch_dst; int64_t num_cols; }; // add two float input eltwise and convert to bf16 : ConvertFP32toBF16(src0 + src1) @@ -545,7 +545,7 @@ class ReduceAdd2bh : public dnnl::impl::cpu::x64::jit_generator { // the prefetch distance is increased to ensure by the time store happens // prefetch has done and no HW prefetcher is triggered args.prefetch_dst = (m + 2 < num_rows) ? (args.dst + 2 * dst_stride) : (args.dst); - + (*this)(&args); } } diff --git a/thirdparty/itt_collector/sea_itt_lib/IttNotifyStdSrc.cpp b/thirdparty/itt_collector/sea_itt_lib/IttNotifyStdSrc.cpp index ee2daf88d79ece..c2372a5f3f7bb6 100644 --- a/thirdparty/itt_collector/sea_itt_lib/IttNotifyStdSrc.cpp +++ b/thirdparty/itt_collector/sea_itt_lib/IttNotifyStdSrc.cpp @@ -1577,15 +1577,13 @@ void FillApiList(__itt_api_info* api_list_ptr) { #define ITT_STUB_IMPL_ORIG(name) ITT_STUB_IMPL(name) #ifdef _DEBUG // dangerous stub that doesn't return anything (even when expected) but records the function call for // statistics sake -# define ITT_STUB_NO_IMPL(fn) \ - if (0 == strcmp("__itt_" ITT_TO_STR(fn), api_list_ptr[i].name)) { \ - struct local { \ - static void stub(...) { \ - CIttFnStat oIttFnStat("NO IMPL:\t" ITT_TO_STR(fn)); \ - } \ - }; \ - *api_list_ptr[i].func_ptr = reinterpret_cast(local::stub); \ - continue; \ +# define ITT_STUB_NO_IMPL(fn) \ + if (0 == strcmp("__itt_" ITT_TO_STR(fn), api_list_ptr[i].name)) { \ + struct local { \ + static void stub(...) { CIttFnStat oIttFnStat("NO IMPL:\t" ITT_TO_STR(fn)); } \ + }; \ + *api_list_ptr[i].func_ptr = reinterpret_cast(local::stub); \ + continue; \ } #else # define ITT_STUB_NO_IMPL(fn)