dpl/dpo: properly load fixed instances flipping at improve_placement

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Author: gudeh

src/dpl/src/Optdp.cpp

@@ -69,11 +69,15 @@ void Opendp::improvePlacement(const int seed,
   importDb();
 
   // importDb() rebuilds the network from scratch; fresh Node objects have
-  // orientation hard coded as R0. The legalizer writes row-correct orientations
-  // to dbInst in its post-flush orient loop, so use this orientation for
-  // DetailedMgr DRC checks
+  // orientation hard coded as R0 with un-rotated master width/height.  The
+  // dbInst already carries the true orientation, so sync every placed cell's
+  // node to it.  Fixed cells (rotated macros, endcaps, tapcells) must be
+  // included: setFixedGridCells() and the DetailedMgr DRC checks read the
+  // node's width/height, so leaving a rotated fixed macro at R0 paints a
+  // wrong-sized footprint and produces phantom overlaps / edge-spacing
+  // violations against it.
   for (const auto& node : network_->getNodes()) {
-    if (node->getType() == Node::CELL && !node->isFixed()) {
+    if (node->getType() == Node::CELL) {
       odb::dbInst* inst = node->getDbInst();
       if (inst && inst->getPlacementStatus().isPlaced()) {
         node->adjustCurrOrient(inst->getOrient());
@@ -107,6 +111,10 @@ void Opendp::improvePlacement(const int seed,
   // improvement.  If it errors or prints a warning when
   // given a legal placement, that likely means there is
   // a bug in my code somewhere.
+  if (debug_observer_) {
+    logger_->report("Pause before \"legalize\" DPO initialization.");
+    debug_observer_->redrawAndPause();
+  }
   ShiftLegalizer lg;
   lg.legalize(mgr);