Merge pull request #9121 from LucasYuki/odb_dbRegionGroupItr_fix2

dpl: Handling filler name conflicts

Author: maliberty

src/dpl/src/FillerPlacement.cpp

@@ -217,10 +217,11 @@ void Opendp::placeRowFillers(GridY row,
       for (dbMaster* master : fillers) {
         std::string inst_name
             = prefix + to_string(row.v) + "_" + to_string(k.v);
-        odb::dbInst* inst = odb::dbInst::create(block_,
-                                                master,
-                                                inst_name.c_str(),
-                                                /* physical_only */ true);
+        odb::dbInst* inst
+            = odb::dbInst::makeUniqueDbInst(block_,
+                                            master,
+                                            inst_name.c_str(),
+                                            /* physical_only */ true);
         DbuX x{core_.xMin() + gridToDbu(k, site_width)};
         DbuY y{core_.yMin() + grid_->gridYToDbu(row)};
         inst->setOrient(orient);

src/odb/include/odb/db.h

@@ -3089,8 +3089,8 @@ class dbInst : public dbObject
   static dbInst* makeUniqueDbInst(dbBlock* block,
                                   dbMaster* master,
                                   const char* name,
-                                  bool physical_only,
-                                  dbModule* target_module);
+                                  bool physical_only = false,
+                                  dbModule* target_module = nullptr);
 
   ///
   /// Create a new instance of child_block in top_block.