[CI] Update clang-format version to 18

Author: aobolensk

.github/workflows/code_style.yml

@@ -9,7 +9,7 @@ permissions: read-all
 
 jobs:
   clang-format:
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     if: ${{ github.repository_owner == 'openvinotoolkit' }}
     permissions:
       pull-requests: write
@@ -19,10 +19,10 @@ jobs:
         with:
           submodules: 'true'
 
-      - name: Install clang-format-15
+      - name: Install clang-format-18
         run: |
           sudo apt update
-          sudo apt --assume-yes install clang-format-15
+          sudo apt --assume-yes install clang-format-18
 
       # Run cmake with extra options to cover as much source code as possible:
       # - -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT to enable codestyle check for ITT collector
@@ -46,7 +46,7 @@ jobs:
               "./temp/*"
 
   clang-format-aarch64:
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     if: ${{ github.repository_owner == 'openvinotoolkit' }}
     permissions:
       pull-requests: write
@@ -55,10 +55,10 @@ jobs:
         with:
           submodules: 'true'
 
-      - name: Install clang-format-15 and cross-compilation dependencies
+      - name: Install clang-format-18 and cross-compilation dependencies
         run: |
           sudo apt update
-          sudo apt --assume-yes install binutils-aarch64-linux-gnu gcc-aarch64-linux-gnu g++-aarch64-linux-gnu scons clang-format-15
+          sudo apt --assume-yes install binutils-aarch64-linux-gnu gcc-aarch64-linux-gnu g++-aarch64-linux-gnu scons clang-format-18
 
       # Run cmake with extra options to cover as much source code as possible:
       # - -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT to enable codestyle check for ITT collector
@@ -82,7 +82,7 @@ jobs:
               "./temp/*"
 
   clang-format-riscv64:
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     if: ${{ github.repository_owner == 'openvinotoolkit' }}
     permissions:
       pull-requests: write
@@ -91,10 +91,10 @@ jobs:
         with:
           submodules: 'true'
 
-      - name: Install clang-format-15 and cross-compilation dependencies
+      - name: Install clang-format-18 and cross-compilation dependencies
         run: |
           sudo apt update
-          sudo apt --assume-yes install gcc-riscv64-linux-gnu g++-riscv64-linux-gnu binutils-riscv64-linux-gnu clang-format-15
+          sudo apt --assume-yes install gcc-riscv64-linux-gnu g++-riscv64-linux-gnu binutils-riscv64-linux-gnu clang-format-18
 
       # Run cmake with extra options to cover as much source code as possible:
       # - -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT to enable codestyle check for ITT collector

cmake/developer_package/clang_format/clang_format.cmake

@@ -3,9 +3,10 @@
 #
 
 if(ENABLE_CLANG_FORMAT)
-    set(CLANG_FORMAT_REQUIRED_VERSION 15 CACHE STRING "Clang-format version to use")
+    set(CLANG_FORMAT_REQUIRED_VERSION 18 CACHE STRING "Clang-format version to use")
     set(CLANG_FORMAT_FILENAME clang-format-${CLANG_FORMAT_REQUIRED_VERSION} clang-format)
     find_host_program(CLANG_FORMAT NAMES ${CLANG_FORMAT_FILENAME} PATHS ENV PATH)
+    message(STATUS "Found clang-format: ${CLANG_FORMAT}")
     if(CLANG_FORMAT)
         execute_process(COMMAND ${CLANG_FORMAT} ${CMAKE_CURRENT_SOURCE_DIR} ARGS --version OUTPUT_VARIABLE CLANG_VERSION)
         if(NOT CLANG_VERSION)

docs/dev/coding_style.md

@@ -2,12 +2,12 @@
 
 ## Coding style
 
-The majority of OpenVINO components use `clang-format-9` for code style check.
+The majority of OpenVINO components use `clang-format-18` for code style check.
 
 The code style is based on the Google Code style with some differences. All the differences are described in the configuration file:
 https://github.com/openvinotoolkit/openvino/blob/69f709028a5f8da596d1d0df9a0101e517c35708/src/.clang-format#L1-L28
 
-To fix the code style on your local machine, you need to install the `clang-format-9` tool and make sure that the CMake option `ENABLE_CLANG_FORMAT` is enabled.
+To fix the code style on your local machine, you need to install the `clang-format-18` tool and make sure that the CMake option `ENABLE_CLANG_FORMAT` is enabled.
 If all dependencies are resolved, you can use the `clang_format_fix_all` target to fix all code style issues.
 
 ## Naming style

samples/cpp/common/utils/include/samples/common.hpp

@@ -563,8 +563,7 @@ static UNUSED void printPerformanceCountsSort(std::vector<ov::ProfilingInfo> per
                 stream << "layerType: ";
                 if (it.node_type.length() >= maxPrintLength) {
                     stream << std::setw(maxPrintLength) << std::left
-                           << it.node_type.substr(0, maxPrintLength - 3) + "..."
-                           << " ";
+                           << it.node_type.substr(0, maxPrintLength - 3) + "..." << " ";
                 } else {
                     stream << std::setw(maxPrintLength) << std::left << it.node_type << " ";
                 }

samples/cpp/common/utils/include/samples/os/windows/w_dirent.h

@@ -41,8 +41,8 @@
 // clang-format on
 
 // Copied from linux libc sys/stat.h:
-#    define S_ISREG(m) (((m)&S_IFMT) == S_IFREG)
-#    define S_ISDIR(m) (((m)&S_IFMT) == S_IFDIR)
+#    define S_ISREG(m) (((m) & S_IFMT) == S_IFREG)
+#    define S_ISDIR(m) (((m) & S_IFMT) == S_IFDIR)
 
 /// @brief structure to store directory names
 struct dirent {

src/bindings/js/node/CONTRIBUTING.md

@@ -43,7 +43,7 @@ all checks have passed. CI checks are composed of both functional tests and code
 
 Remember to follow [our codestyle](../docs/CODESTYLE.md).
 By following the provided guide and using an automotive code style checking tool, like
-**eslint** and **clang-format-9**, you will save some time and help with the code review of proposed changes.
+**eslint** and **clang-format-18**, you will save some time and help with the code review of proposed changes.
 
 
 ## Description of the Pull Request

src/bindings/js/node/src/preprocess/input_info.cpp

@@ -8,7 +8,7 @@
 #include "node/include/preprocess/input_tensor_info.hpp"
 #include "node/include/preprocess/preprocess_steps.hpp"
 
-InputInfo::InputInfo(const Napi::CallbackInfo& info) : Napi::ObjectWrap<InputInfo>(info), _input_info(nullptr){};
+InputInfo::InputInfo(const Napi::CallbackInfo& info) : Napi::ObjectWrap<InputInfo>(info), _input_info(nullptr) {};
 
 Napi::Function InputInfo::get_class_constructor(Napi::Env env) {
     return DefineClass(env,

src/bindings/js/node/src/preprocess/input_model_info.cpp

@@ -8,7 +8,7 @@
 
 InputModelInfo::InputModelInfo(const Napi::CallbackInfo& info)
     : Napi::ObjectWrap<InputModelInfo>(info),
-      _model_info(nullptr){};
+      _model_info(nullptr) {};
 
 Napi::Function InputModelInfo::get_class_constructor(Napi::Env env) {
     return DefineClass(env, "InputModelInfo", {InstanceMethod("setLayout", &InputModelInfo::set_layout)});

src/bindings/js/node/src/preprocess/input_tensor_info.cpp

@@ -8,7 +8,7 @@
 
 InputTensorInfo::InputTensorInfo(const Napi::CallbackInfo& info)
     : Napi::ObjectWrap<InputTensorInfo>(info),
-      _tensor_info(nullptr){};
+      _tensor_info(nullptr) {};
 
 Napi::Function InputTensorInfo::get_class_constructor(Napi::Env env) {
     return DefineClass(env,

src/bindings/js/node/src/preprocess/output_info.cpp

@@ -6,7 +6,7 @@
 #include "node/include/errors.hpp"
 #include "node/include/preprocess/output_tensor_info.hpp"
 
-OutputInfo::OutputInfo(const Napi::CallbackInfo& info) : Napi::ObjectWrap<OutputInfo>(info), _output_info(nullptr){};
+OutputInfo::OutputInfo(const Napi::CallbackInfo& info) : Napi::ObjectWrap<OutputInfo>(info), _output_info(nullptr) {};
 
 Napi::Function OutputInfo::get_class_constructor(Napi::Env env) {
     return DefineClass(env, "OutputInfo", {InstanceMethod("tensor", &OutputInfo::tensor)});

src/bindings/js/node/src/preprocess/output_tensor_info.cpp

@@ -8,7 +8,7 @@
 
 OutputTensorInfo::OutputTensorInfo(const Napi::CallbackInfo& info)
     : Napi::ObjectWrap<OutputTensorInfo>(info),
-      _tensor_info(nullptr){};
+      _tensor_info(nullptr) {};
 
 Napi::Function OutputTensorInfo::get_class_constructor(Napi::Env env) {
     return DefineClass(env,

src/bindings/js/node/src/preprocess/preprocess_steps.cpp

@@ -5,7 +5,7 @@
 
 PreProcessSteps::PreProcessSteps(const Napi::CallbackInfo& info)
     : Napi::ObjectWrap<PreProcessSteps>(info),
-      _preprocess_info(nullptr){};
+      _preprocess_info(nullptr) {};
 
 Napi::Function PreProcessSteps::get_class_constructor(Napi::Env env) {
     return DefineClass(env, "PreProcessSteps", {InstanceMethod("resize", &PreProcessSteps::resize)});

src/common/transformations/include/transformations/common_optimizations/simplify_shape_of_sub_graph.hpp

@@ -42,7 +42,7 @@ class ov::pass::GroupedGatherElimination : public ov::pass::MatcherPass {
 class ov::pass::SimplifyShapeOfSubGraph : public ov::pass::ModelPass {
 public:
     OPENVINO_MODEL_PASS_RTTI("SimplifyShapeOfSubGraph");
-    explicit SimplifyShapeOfSubGraph(bool use_shapes = true) : m_use_shapes(use_shapes){};
+    explicit SimplifyShapeOfSubGraph(bool use_shapes = true) : m_use_shapes(use_shapes) {};
     bool run_on_model(const std::shared_ptr<ov::Model>& m) override;
 
 private:

src/common/transformations/src/transformations/common_optimizations/lin_op_sequence_fusion.cpp

@@ -23,7 +23,7 @@ const auto is_eltwise_supported_type = [](const Output<Node>& output) -> bool {
     const auto is_single_output = pass::pattern::consumers_count(1);
     return is_single_output(output) && output.get_node()->has_evaluate();
 };
-}
+}  // namespace
 
 ov::pass::AddMultiplyFusion::AddMultiplyFusion() {
     MATCHER_SCOPE(AddMultiplyFusion);

src/common/transformations/src/transformations/op_conversions/einsum_decomposition.cpp

@@ -1326,7 +1326,7 @@ ov::pass::EinsumDecomposition::EinsumDecomposition() {
         fix_inputs_with_0d_ellipsis(input_nodes, input_subscripts, output_subscript, subgraph_nodes);
 
         // contract inputs by Einsum until just one is remained
-        for (auto const& inds_pair : einsum_path) {
+        for (const auto& inds_pair : einsum_path) {
             contract_two_inputs(input_nodes,
                                 input_subscripts,
                                 output_subscript,

src/common/transformations/tests/utils/compare_functions_test.cpp

@@ -675,7 +675,7 @@ const auto createU1ConstantFunc = [](const Shape& s, const uint8_t* data) {
 
     return std::make_shared<ov::Model>(OutputVector{c}, ParameterVector{});
 };
-}
+}  // namespace
 
 TEST(TransformationTests, ConstantComparison_ElementTypeU1_Positive_1stbit) {
     const Shape shape{1};

src/common/util/src/file_util.cpp

@@ -37,7 +37,7 @@
 #    endif
 // Copied from linux libc sys/stat.h:
 #    if !defined(__MINGW32__) && !defined(__MINGW64__)
-#        define S_ISDIR(m) (((m)&S_IFMT) == S_IFDIR)
+#        define S_ISDIR(m) (((m) & S_IFMT) == S_IFDIR)
 #    endif
 #else
 #    include <dirent.h>

src/core/include/openvino/core/any.hpp

@@ -38,8 +38,8 @@ struct Read;
 template <class T>
 struct Readable {
     template <class U>
-    static auto test(U*)
-        -> decltype(std::declval<Read<U>>()(std::declval<std::istream&>(), std::declval<U&>()), std::true_type()) {
+    static auto test(U*) -> decltype(std::declval<Read<U>>()(std::declval<std::istream&>(), std::declval<U&>()),
+                                     std::true_type()) {
         return {};
     }
     template <typename>

src/core/include/openvino/core/extension.hpp

@@ -50,6 +50,8 @@ void OV_CREATE_EXTENSION(std::vector<ov::Extension::Ptr>& ext);
  *
  * @param ext of extensions
  */
-#define OPENVINO_CREATE_EXTENSIONS(extensions)                                               \
-    OPENVINO_EXTENSION_C_API void OV_CREATE_EXTENSION(std::vector<ov::Extension::Ptr>& ext); \
-    OPENVINO_EXTENSION_C_API void OV_CREATE_EXTENSION(std::vector<ov::Extension::Ptr>& ext) { ext = extensions; }
+#define OPENVINO_CREATE_EXTENSIONS(extensions)                                                \
+    OPENVINO_EXTENSION_C_API void OV_CREATE_EXTENSION(std::vector<ov::Extension::Ptr>& ext);  \
+    OPENVINO_EXTENSION_C_API void OV_CREATE_EXTENSION(std::vector<ov::Extension::Ptr>& ext) { \
+        ext = extensions;                                                                     \
+    }

src/core/include/openvino/core/rtti.hpp

@@ -19,7 +19,9 @@
         type_info_static.hash();                                                          \
         return type_info_static;                                                          \
     }                                                                                     \
-    const ::ov::DiscreteTypeInfo& get_type_info() const override { return get_type_info_static(); }
+    const ::ov::DiscreteTypeInfo& get_type_info() const override {                        \
+        return get_type_info_static();                                                    \
+    }
 
 #define _OPENVINO_RTTI_WITH_TYPE_VERSION_PARENT(TYPE_NAME, VERSION_NAME, PARENT_CLASS) \
     _OPENVINO_RTTI_WITH_TYPE_VERSIONS_PARENT(TYPE_NAME, VERSION_NAME, PARENT_CLASS)
@@ -32,7 +34,9 @@
         type_info_static.hash();                                                               \
         return type_info_static;                                                               \
     }                                                                                          \
-    const ::ov::DiscreteTypeInfo& get_type_info() const override { return get_type_info_static(); }
+    const ::ov::DiscreteTypeInfo& get_type_info() const override {                             \
+        return get_type_info_static();                                                         \
+    }
 
 /// Helper macro that puts necessary declarations of RTTI block inside a class definition.
 /// Should be used in the scope of class that requires type identification besides one provided by
@@ -100,7 +104,9 @@
         type_info_static.hash();                                                          \
         return type_info_static;                                                          \
     }                                                                                     \
-    virtual const ::ov::DiscreteTypeInfo& get_type_info() const { return get_type_info_static(); }
+    virtual const ::ov::DiscreteTypeInfo& get_type_info() const {                         \
+        return get_type_info_static();                                                    \
+    }
 
 /// Helper macro for base (without rtti parrent) class that provides RTTI block definition.
 /// It's a two parameter version of OPENVINO_RTTI macro, without PARENT_CLASS.

src/core/include/openvino/core/type/bfloat16.hpp

@@ -20,18 +20,19 @@ namespace ov {
 class OPENVINO_API bfloat16 {
 public:
     bfloat16() = default;
-    bfloat16(float value) : m_value {
+    bfloat16(float value)
+        : m_value{
 #if defined ROUND_MODE_TO_NEAREST
-        round_to_nearest(value)
+              round_to_nearest(value)
 #elif defined ROUND_MODE_TO_NEAREST_EVEN
-        round_to_nearest_even(value)
+              round_to_nearest_even(value)
 #elif defined ROUND_MODE_TRUNCATE
-        truncate(value)
+              truncate(value)
 #else
 #    error "ROUNDING_MODE must be one of ROUND_MODE_TO_NEAREST, ROUND_MODE_TO_NEAREST_EVEN, or ROUND_MODE_TRUNCATE"
 #endif
+          } {
     }
-    {}
 
     template <typename I>
     explicit bfloat16(I value) : m_value{bfloat16{static_cast<float>(value)}.m_value} {}

src/core/include/openvino/pass/pattern/op/optional.hpp

@@ -54,7 +54,7 @@ class OPENVINO_API Optional : public Pattern {
     /// \param patterns The pattern to match a graph.
     Optional(const std::vector<DiscreteTypeInfo>& type_infos, const OutputVector& inputs = {})
         : Pattern(inputs),
-          optional_types(type_infos){};
+          optional_types(type_infos) {};
 
     template <typename TPredicate>
     Optional(const std::vector<DiscreteTypeInfo>& type_infos, const OutputVector& inputs, const TPredicate& pred)

src/core/reference/src/op/experimental_detectron_roi_feature_extractor.cpp

@@ -275,8 +275,8 @@ void ROIAlignForward(const int64_t nthreads,
 
                     top_data[index] = output_val;
                 }  // for pw
-            }      // for ph
-        }          // for c
+            }  // for ph
+        }  // for c
     }
 }
 }  // namespace

src/core/reference/src/utils/coordinate_index.cpp

@@ -14,7 +14,7 @@ std::size_t coordinate_index(const Coordinate& c, const Shape& s) {
     }
     std::size_t index = 0;
     std::size_t stride = 1;
-    std::size_t const padding = c.size() - s.size();
+    const std::size_t padding = c.size() - s.size();
 
     for (std::size_t axis = s.size(); axis-- > 0;) {
         if (s[axis] > 1) {

src/core/reference/src/utils/philox_generator.cpp

@@ -64,7 +64,7 @@ void PhiloxGenerator::set_operator_seed(const uint64_t op_seed) {
 }
 
 // ====== MockPhiloxGenerator functions ======
-MockPhiloxGenerator::MockPhiloxGenerator() : PhiloxGenerator(op::PhiloxAlignment::MOCK, 0, 0, {0, 0}){};
+MockPhiloxGenerator::MockPhiloxGenerator() : PhiloxGenerator(op::PhiloxAlignment::MOCK, 0, 0, {0, 0}) {};
 
 std::pair<uint64_t, uint64_t> MockPhiloxGenerator::get_next_state() {
     return get_previous_state();

src/core/src/itt.hpp

@@ -50,9 +50,11 @@ OV_CC_DOMAINS(ov_opset);
 #    define INSERT_OP(opset_name, op_name, op_namespace) opset.insert<op_namespace::op_name>()
 #endif
 
-#define OPENVINO_TYPE_CASE(region, a, ...)                                                                \
-    case ov::element::Type_t::a: {                                                                        \
-        OV_SCOPE(ov_op, OV_PP_CAT3(region, _, a)) { rc = evaluate<ov::element::Type_t::a>(__VA_ARGS__); } \
+#define OPENVINO_TYPE_CASE(region, a, ...)                      \
+    case ov::element::Type_t::a: {                              \
+        OV_SCOPE(ov_op, OV_PP_CAT3(region, _, a)) {             \
+            rc = evaluate<ov::element::Type_t::a>(__VA_ARGS__); \
+        }                                                       \
     } break
 
 #define OPENVINO_2_TYPES_CASE(region, a, b, ...)                                \
@@ -62,7 +64,9 @@ OV_CC_DOMAINS(ov_opset);
         }                                                                       \
     } break
 
-#define OPENVINO_COPY_TENSOR(region, a, ...)                                                                 \
-    case ov::element::Type_t::a: {                                                                           \
-        OV_SCOPE(ov_op, OV_PP_CAT3(region, _, a)) { rc = copy_tensor<ov::element::Type_t::a>(__VA_ARGS__); } \
+#define OPENVINO_COPY_TENSOR(region, a, ...)                       \
+    case ov::element::Type_t::a: {                                 \
+        OV_SCOPE(ov_op, OV_PP_CAT3(region, _, a)) {                \
+            rc = copy_tensor<ov::element::Type_t::a>(__VA_ARGS__); \
+        }                                                          \
     } break

src/core/src/opsets/opset.cpp

@@ -97,9 +97,7 @@ std::string ov::OpSet::to_upper_name(const std::string& name) {
 
 const std::map<std::string, std::function<const ov::OpSet&()>>& ov::get_available_opsets() {
 #define _OPENVINO_REG_OPSET(OPSET) \
-    {                              \
-#        OPSET, ov::get_##OPSET    \
-    }
+    { #OPSET, ov::get_##OPSET }
     const static std::map<std::string, std::function<const ov::OpSet&()>> opset_map = {_OPENVINO_REG_OPSET(opset1),
                                                                                        _OPENVINO_REG_OPSET(opset2),
                                                                                        _OPENVINO_REG_OPSET(opset3),

src/core/src/pattern/op/predicate.cpp

@@ -8,11 +8,11 @@
 
 namespace ov::pass::pattern {
 
-PatternSymbolValue::PatternSymbolValue() : m_value(){};
-PatternSymbolValue::PatternSymbolValue(const std::shared_ptr<ov::Symbol>& s) : m_value(s){};
-PatternSymbolValue::PatternSymbolValue(int64_t i) : m_value(i){};
-PatternSymbolValue::PatternSymbolValue(double d) : m_value(d){};
-PatternSymbolValue::PatternSymbolValue(const std::vector<PatternSymbolValue>& g) : m_value(g){};
+PatternSymbolValue::PatternSymbolValue() : m_value() {};
+PatternSymbolValue::PatternSymbolValue(const std::shared_ptr<ov::Symbol>& s) : m_value(s) {};
+PatternSymbolValue::PatternSymbolValue(int64_t i) : m_value(i) {};
+PatternSymbolValue::PatternSymbolValue(double d) : m_value(d) {};
+PatternSymbolValue::PatternSymbolValue(const std::vector<PatternSymbolValue>& g) : m_value(g) {};
 
 bool PatternSymbolValue::is_dynamic() const {
     return is_valid() && m_value.is<std::shared_ptr<ov::Symbol>>();