Merge pull request #6077 from The-OpenROAD-Project-staging/sta-reset-…

…on-sigtype

odb/dbSta: reset sta on dbBTerm::setSigType

src/dbSta/src/dbSta.cc

@@ -150,6 +150,7 @@ class dbStaCbk : public dbBlockCallBackObj
   void inDbBTermCreate(dbBTerm*) override;
   void inDbBTermDestroy(dbBTerm* bterm) override;
   void inDbBTermSetIoType(dbBTerm* bterm, const dbIoType& io_type) override;
+  void inDbBTermSetSigType(dbBTerm* bterm, const dbSigType& sig_type) override;
 
  private:
   dbSta* sta_;
@@ -882,6 +883,13 @@ void dbStaCbk::inDbBTermSetIoType(dbBTerm* bterm, const dbIoType& io_type)
   sta_->getDbNetwork()->setTopPortDirection(bterm, io_type);
 }
 
+void dbStaCbk::inDbBTermSetSigType(dbBTerm* bterm, const dbSigType& sig_type)
+{
+  // sta can't handle such changes, see OpenROAD#6025 so just reset the whole
+  // thing.
+  sta_->networkChanged();
+}
+
 ////////////////////////////////////////////////////////////////
 
 // Highlight path in the gui.

src/odb/include/odb/dbBlockCallBackObj.h

@@ -54,6 +54,7 @@ class dbPlacementStatus;
 class dbObstruction;
 class dbRegion;
 class dbRow;
+class dbSigType;
 class dbSBox;
 class dbSWire;
 class dbMarker;
@@ -107,6 +108,7 @@ class dbBlockCallBackObj
   virtual void inDbBTermPreDisconnect(dbBTerm*) {}
   virtual void inDbBTermPostDisConnect(dbBTerm*, dbNet*) {}
   virtual void inDbBTermSetIoType(dbBTerm*, const dbIoType&) {}
+  virtual void inDbBTermSetSigType(dbBTerm*, const dbSigType&) {}
   // dbBTerm End
 
   // dbBPin Start

src/odb/src/db/dbBTerm.cpp

@@ -365,6 +365,10 @@ void dbBTerm::setSigType(dbSigType type)
     block->_journal->updateField(
         this, _dbBTerm::FLAGS, prev_flags, flagsToUInt(bterm));
   }
+
+  for (auto callback : block->_callbacks) {
+    callback->inDbBTermSetSigType(this, type);
+  }
 }
 
 dbSigType dbBTerm::getSigType()