Change visit_attributes default behavior, operation by default supports visit attributes (#14470)

src/core/include/openvino/core/node.hpp

@@ -194,9 +194,7 @@ public:
 
     virtual ~Node();
 
-    virtual bool visit_attributes(AttributeVisitor&) {
+    virtual bool visit_attributes(AttributeVisitor&);
-        return false;
-    }
     /// \returns the autobroadcasr spec
     virtual const ov::op::AutoBroadcastSpec& get_autob() const;
 

src/core/include/openvino/op/abs.hpp

@@ -19,9 +19,6 @@ public:
     OPENVINO_OP("Abs", "opset1", util::UnaryElementwiseArithmetic);
     /// \brief Constructs an absolute value operation.
     Abs() = default;
-    bool visit_attributes(AttributeVisitor&) override {
-        return true;
-    }
     /// \brief Constructs an absolute value operation.
     ///
     /// \param arg Output that produces the input tensor.<br>

src/core/include/openvino/op/acos.hpp

@@ -27,9 +27,6 @@ public:
     /// Output `[d1, ...]`
     ///
     Acos(const Output<Node>& arg);
-    bool visit_attributes(AttributeVisitor&) override {
-        return true;
-    }
     std::shared_ptr<Node> clone_with_new_inputs(const OutputVector& new_args) const override;
     OPENVINO_SUPPRESS_DEPRECATED_START
     bool evaluate(const HostTensorVector& outputs, const HostTensorVector& inputs) const override;

src/core/include/openvino/op/acosh.hpp

@@ -30,9 +30,6 @@ public:
     Acosh(const Output<Node>& arg);
 
     std::shared_ptr<Node> clone_with_new_inputs(const OutputVector& new_args) const override;
-    bool visit_attributes(AttributeVisitor&) override {
-        return true;
-    }
     OPENVINO_SUPPRESS_DEPRECATED_START
     bool evaluate(const HostTensorVector& outputs, const HostTensorVector& inputs) const override;
     OPENVINO_SUPPRESS_DEPRECATED_END

src/core/include/openvino/op/adaptive_avg_pool.hpp

@@ -30,7 +30,6 @@ public:
     AdaptiveAvgPool(const Output<Node>& data, const Output<Node>& output_shape);
 
     void validate_and_infer_types() override;
-    bool visit_attributes(AttributeVisitor& visitor) override;
 
     std::shared_ptr<Node> clone_with_new_inputs(const OutputVector& new_args) const override;
 };

src/core/include/openvino/op/logical_not.hpp

@@ -21,7 +21,6 @@ public:
     /// \param arg Node that produces the input tensor.
     LogicalNot(const Output<Node>& arg);
 
-    bool visit_attributes(AttributeVisitor& visitor) override;
     void validate_and_infer_types() override;
 
     std::shared_ptr<Node> clone_with_new_inputs(const OutputVector& new_args) const override;

src/core/src/node.cpp

@@ -844,6 +844,10 @@ bool ov::Node::is_const_fold_disabled() const {
     return ov::pass::constant_folding_is_disabled(this);
 }
 
+bool ov::Node::visit_attributes(AttributeVisitor&) {
+    return true;
+}
+
 namespace ov {
 AttributeAdapter<std::shared_ptr<Node>>::AttributeAdapter(std::shared_ptr<Node>& value) : m_ref(value) {}
 

src/core/src/op/adaptive_avg_pool.cpp

@@ -17,11 +17,6 @@ op::v8::AdaptiveAvgPool::AdaptiveAvgPool(const Output<Node>& data, const Output<
     constructor_validate_and_infer_types();
 }
 
-bool op::v8::AdaptiveAvgPool::visit_attributes(AttributeVisitor& visitor) {
-    OV_OP_SCOPE(v8_AdaptiveAvgPool_visit_attributes);
-    return true;
-}
-
 void op::v8::AdaptiveAvgPool::validate_and_infer_types() {
     OV_OP_SCOPE(v8_AdaptiveAvgPool_validate_and_infer_types);
 

src/core/src/op/logical_not.cpp

@@ -17,11 +17,6 @@ op::v1::LogicalNot::LogicalNot(const Output<Node>& arg) : Op({arg}) {
     constructor_validate_and_infer_types();
 }
 
-bool ngraph::op::v1::LogicalNot::visit_attributes(AttributeVisitor& visitor) {
-    OV_OP_SCOPE(v1_LogicalNot_visit_attributes);
-    return true;
-}
-
 void op::v1::LogicalNot::validate_and_infer_types() {
     OV_OP_SCOPE(v1_LogicalNot_validate_and_infer_types);
     const auto& element_type = get_input_element_type(0);