[FFI] Cleanup visit_attrs attribute after refactor (#18112)

This PR is a cleanup pr after we removed VisitAttrs, the attribute is no longer
necessary

Author: tqchen

include/tvm/arith/analyzer.h

@@ -94,8 +94,6 @@ class ConstIntBoundNode : public Object {
         .def_ro("max_value", &ConstIntBoundNode::max_value);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const ConstIntBoundNode* other, SEqualReducer equal) const {
     return equal(min_value, other->min_value) && equal(max_value, other->max_value);
   }
@@ -220,8 +218,6 @@ class ModularSetNode : public Object {
         .def_ro("base", &ModularSetNode::base);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const ModularSetNode* other, SEqualReducer equal) const {
     return equal(coeff, other->coeff) && equal(base, other->base);
   }

include/tvm/arith/iter_affine_map.h

@@ -66,7 +66,6 @@ namespace arith {
  */
 class IterMapExprNode : public PrimExprNode {
  public:
-  static constexpr bool _type_has_method_visit_attrs = false;
   static constexpr const char* _type_key = "arith.IterMapExpr";
   static constexpr const uint32_t _type_child_slots = 2;
   TVM_DECLARE_BASE_OBJECT_INFO(IterMapExprNode, PrimExprNode);
@@ -106,8 +105,6 @@ class IterMarkNode : public Object {
         .def_ro("extent", &IterMarkNode::extent);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const IterMarkNode* other, SEqualReducer equal) const {
     equal->MarkGraphNode();
     return equal(source, other->source) && equal(extent, other->extent);
@@ -167,8 +164,6 @@ class IterSplitExprNode : public IterMapExprNode {
         .def_ro("scale", &IterSplitExprNode::scale);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const IterSplitExprNode* other, SEqualReducer equal) const {
     return equal(source, other->source) && equal(lower_factor, other->lower_factor) &&
            equal(extent, other->extent) && equal(scale, other->scale);
@@ -235,8 +230,6 @@ class IterSumExprNode : public IterMapExprNode {
         .def_ro("base", &IterSumExprNode::base);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const IterSumExprNode* other, SEqualReducer equal) const {
     return equal(args, other->args) && equal(base, other->base);
   }
@@ -307,8 +300,6 @@ class IterMapResultNode : public Object {
         .def_ro("padding_predicate", &IterMapResultNode::padding_predicate);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   static constexpr const char* _type_key = "arith.IterMapResult";
   TVM_DECLARE_FINAL_OBJECT_INFO(IterMapResultNode, Object);
 };

include/tvm/ir/attrs.h

@@ -82,7 +82,7 @@ class AttrFieldInfoNode : public Object {
   }
 
   static constexpr const char* _type_key = "ir.AttrFieldInfo";
-  static constexpr bool _type_has_method_visit_attrs = false;
+
   static constexpr bool _type_has_method_sequal_reduce = false;
   static constexpr bool _type_has_method_shash_reduce = false;
   TVM_DECLARE_FINAL_OBJECT_INFO(AttrFieldInfoNode, Object);

include/tvm/ir/diagnostic.h

@@ -74,8 +74,6 @@ class DiagnosticNode : public Object {
         .def_ro("message", &DiagnosticNode::message);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const DiagnosticNode* other, SEqualReducer equal) const {
     return equal(this->level, other->level) && equal(this->span, other->span) &&
            equal(this->message, other->message);
@@ -174,8 +172,6 @@ class DiagnosticRendererNode : public Object {
     refl::ObjectDef<DiagnosticRendererNode>().def_ro("renderer", &DiagnosticRendererNode::renderer);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   static constexpr const char* _type_key = "DiagnosticRenderer";
   TVM_DECLARE_FINAL_OBJECT_INFO(DiagnosticRendererNode, Object);
 };
@@ -214,8 +210,6 @@ class DiagnosticContextNode : public Object {
         .def_ro("diagnostics", &DiagnosticContextNode::diagnostics);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const DiagnosticContextNode* other, SEqualReducer equal) const {
     return equal(module, other->module) && equal(diagnostics, other->diagnostics);
   }

include/tvm/ir/env_func.h

@@ -54,8 +54,6 @@ class EnvFuncNode : public Object {
     refl::ObjectDef<EnvFuncNode>().def_ro("name", &EnvFuncNode::name);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   bool SEqualReduce(const EnvFuncNode* other, SEqualReducer equal) const {
     // name uniquely identifies the env function.
     return name == other->name;

include/tvm/ir/global_info.h

@@ -78,8 +78,6 @@ class VDeviceNode : public GlobalInfoNode {
         .def_ro("memory_scope", &VDeviceNode::memory_scope);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   TVM_DLL bool SEqualReduce(const VDeviceNode* other, SEqualReducer equal) const {
     return equal(target, other->target) && equal(vdevice_id, other->vdevice_id) &&
            equal(memory_scope, other->memory_scope);
@@ -114,8 +112,6 @@ class DummyGlobalInfoNode : public GlobalInfoNode {
     refl::ObjectDef<DummyGlobalInfoNode>();
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   static constexpr const char* _type_key = "ir.DummyGlobalInfo";
 
   TVM_DLL bool SEqualReduce(const DummyGlobalInfoNode* other, SEqualReducer equal) const {

include/tvm/ir/global_var_supply.h

@@ -81,8 +81,6 @@ class GlobalVarSupplyNode : public Object {
     refl::ObjectDef<GlobalVarSupplyNode>();
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   /*! \brief The NameSupply used to generate unique name hints to GlobalVars. */
   NameSupply name_supply_;
 

include/tvm/ir/instrument.h

@@ -142,8 +142,6 @@ class PassInstrumentNode : public Object {
     refl::ObjectDef<PassInstrumentNode>().def_ro("name", &PassInstrumentNode::name);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   static constexpr const char* _type_key = "instrument.PassInstrument";
   TVM_DECLARE_BASE_OBJECT_INFO(PassInstrumentNode, Object);
 };

include/tvm/ir/module.h

@@ -140,8 +140,6 @@ class IRModuleNode : public Object {
         .def_ro("global_infos", &IRModuleNode::global_infos);
   }
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   TVM_DLL bool SEqualReduce(const IRModuleNode* other, SEqualReducer equal) const;
 
   TVM_DLL void SHashReduce(SHashReducer hash_reduce) const;

include/tvm/ir/name_supply.h

@@ -81,8 +81,6 @@ class NameSupplyNode : public Object {
    */
   bool ContainsName(const String& name, bool add_prefix = true);
 
-  static constexpr bool _type_has_method_visit_attrs = false;
-
   // Prefix for all GlobalVar names. It can be empty.
   std::string prefix_;