Merge pull request #8065 from The-OpenROAD-Project-staging/secure-remove_buffer-fix

Hier: Resolved hang and crash in asap7/swerv_wrapper design

Author: maliberty

src/odb/src/db/dbBTerm.cpp

@@ -395,6 +395,11 @@ void dbBTerm::connect(dbNet* net_)
   _dbNet* net = (_dbNet*) net_;
   _dbBlock* block = (_dbBlock*) net->getOwner();
 
+  // Same net. Nothing to connect.
+  if (bterm->_net == net_->getId()) {
+    return;
+  }
+
   if (net->_flags._dont_touch) {
     net->getLogger()->error(utl::ODB,
                             377,

src/rsz/src/Resizer.cc

@@ -371,6 +371,10 @@ void Resizer::init()
 // remove all buffers if no buffers are specified
 void Resizer::removeBuffers(sta::InstanceSeq insts)
 {
+  // Unlike Resizer::bufferInputs(), init() call is not needed here.
+  // init() call performs STA levelization, but removeBuffers() does not need
+  // timing information. So initBlock(), a light version of init(), is
+  // sufficient.
   initBlock();
   // Disable incremental timing.
   graph_delay_calc_->delaysInvalid();