Remove redundant user*ClearTree calls

src/V3Active.cpp

@@ -125,8 +125,6 @@ public:
     // METHODS
     void begin() {
         // Start a new if/else tracking graph
-        // See NODE STATE comment in ActiveLatchCheckVisitor
-        AstNode::user1ClearTree();
         m_curVertexp = new LatchDetectGraphVertex{this, "ROOT"};
     }
     // Clear out userp field of referenced outputs on destruction

src/V3LifePost.cpp

@@ -252,8 +252,6 @@ private:
 
     // VISITORS
     void visit(AstTopScope* nodep) override {
-        AstNode::user4ClearTree();  // user4p() used on entire tree
-
         // First, build maps of every location (mtask and sequence
         // within the mtask) where each varscope is read, and written.
         iterateChildren(nodep);

src/V3LinkCells.cpp

@@ -160,7 +160,6 @@ private:
 
     // VISITs
     void visit(AstNetlist* nodep) override {
-        AstNode::user1ClearTree();
         readModNames();
         iterateChildren(nodep);
         // Find levels in graph

src/V3Task.cpp

@@ -273,8 +273,6 @@ public:
     // CONSTRUCTORS
     explicit TaskStateVisitor(AstNetlist* nodep) {
         m_curVxp = new TaskCodeVertex{&m_callGraph};
-        AstNode::user3ClearTree();
-        AstNode::user4ClearTree();
         //
         iterate(nodep);
         //