Merge NamePool and RootNamePool into a single type (#7797)

* Initial plan

* Merge NamePool and RootNamePool into single NamePool class

Co-authored-by: csyonghe <2652293+csyonghe@users.noreply.github.com>

* Remove unnecessary comment from slang-fiddle-scrape.cpp

Co-authored-by: Theresa Foley <tangent-vector@users.noreply.github.com>

* Address review feedback: initialize namePool to nullptr and remove unnecessary comments

Co-authored-by: Theresa Foley <tangent-vector@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: csyonghe <2652293+csyonghe@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Theresa Foley <tangent-vector@users.noreply.github.com>
Co-authored-by: Yong He <yonghe@outlook.com>

1 files changed, 0 insertions, 3 deletions

diff --git a/tools/slang-cpp-parser/unit-test.cpp b/tools/slang-cpp-parser/unit-test.cpp
index 70851fd7c..41420578e 100644
--- a/tools/slang-cpp-parser/unit-test.cpp
+++ b/tools/slang-cpp-parser/unit-test.cpp
@@ -25,13 +25,10 @@ struct TestState
 
         m_sink.init(&m_sourceManager, Lexer::sourceLocationLexer);
 
-        m_namePool.setRootNamePool(&m_rootNamePool);
-
         // We don't require marker
         m_options.m_requireMark = false;
     }
 
-    RootNamePool m_rootNamePool;
     Options m_options;
     SourceManager m_sourceManager;
     DiagnosticSink m_sink;