Optimized with a new dbNet::isInternalTo() API

Signed-off-by: Jaehyun Kim <[email protected]>

Author: jhkim-pii

src/odb/include/odb/db.h

@@ -2504,6 +2504,13 @@ class dbNet : public dbObject
   ///
   void renameWithModNetInHighestHier();
 
+  ///
+  /// Check if this net is internal to the given module.
+  /// A net is internal if all its iterms belong to instances within the module
+  /// and it has no bterms.
+  ///
+  bool isInternalTo(dbModule* module) const;
+
   ///
   /// Check issues such as multiple drivers, no driver, or dangling net
   ///

src/odb/src/db/dbModule.cpp

@@ -878,23 +878,20 @@ void _dbModule::copyModuleInsts(dbModule* old_module,
       //         net_name = u0/_001_        <-- External net crossing module
       //                                        boundary.
       std::string old_net_name = old_net->getName();
-      if (old_module->getModInst() != nullptr) {
-        std::string modinst_name = old_module->getHierarchicalName();
-        if (old_net_name.compare(0, modinst_name.length(), modinst_name) != 0) {
-          // Skip external net crossing module boundary.
-          // It will be connected later.
-          debugPrint(logger,
-                     utl::ODB,
-                     "replace_design",
-                     3,
-                     "Skip: dbNet '{}'. Hierarchy_prefix='{}'\n",
-                     old_net_name,
-                     modinst_name);
-          continue;
-        }
+      if (old_net->isInternalTo(old_module) == false) {
+        // Skip external net crossing module boundary.
+        // It will be connected later.
+        debugPrint(logger,
+                   utl::ODB,
+                   "replace_design",
+                   3,
+                   "    Skip: non-internal dbNet '{}' of old_module '{}'.\n",
+                   old_net_name,
+                   old_module->getHierarchicalName());
+        continue;
       }
-      new_net_name += block->getBaseName(old_net_name.c_str());
 
+      new_net_name += block->getBaseName(old_net_name.c_str());
       auto it = new_net_name_map.find(new_net_name);
       if (it != new_net_name_map.end()) {
         // Connect to an existing local net

src/odb/src/db/dbNet.cpp

@@ -2599,4 +2599,21 @@ void dbNet::renameWithModNetInHighestHier()
   }
 }
 
+bool dbNet::isInternalTo(dbModule* module) const
+{
+  // If it's connected to any top-level ports (BTerms), it's not internal.
+  if (!getBTerms().empty()) {
+    return false;
+  }
+
+  // Check all instance terminals (ITerms) it's connected to.
+  for (dbITerm* iterm : getITerms()) {
+    if (iterm->getInst()->getModule() != module) {
+      return false;
+    }
+  }
+
+  return true;
+}
+
 }  // namespace odb