mpl2: comments

Signed-off-by: Arthur Koucher <arthurckoucher@gmail.com>

src/mpl2/src/hier_rtlmp.cpp

@@ -6195,6 +6195,8 @@ void HierRTLMP::updateMacrosOnDb()
   }
 }
 
+// We don't lock the macros here, because we'll attempt to improve
+// orientation next.
 void HierRTLMP::updateMacroOnDb(const HardMacro* hard_macro)
 {
   odb::dbInst* inst = hard_macro->getInst();
@@ -6206,11 +6208,10 @@ void HierRTLMP::updateMacroOnDb(const HardMacro* hard_macro)
   const int x = block_->micronsToDbu(hard_macro->getRealX());
   const int y = block_->micronsToDbu(hard_macro->getRealY());
 
+  // Orientation must be set before location so we don't end up flipping
+  // and misplacing the macro.
   inst->setOrient(hard_macro->getOrientation());
   inst->setLocation(x, y);
-
-  // We don't lock the macros here, because we'll attempt to improve
-  // orientation next.
   inst->setPlacementStatus(odb::dbPlacementStatus::PLACED);
 }
 

src/mpl2/src/rtl_mp.cpp

@@ -148,6 +148,8 @@ void MacroPlacer2::placeMacro(odb::dbInst* inst,
                    "place macro outside of the core.");
   }
 
+  // Orientation must be set before location so we don't end up flipping
+  // and misplacing the macro.
   inst->setOrient(orientation);
   inst->setLocation(x1, y1);