gpl: treat VISUAL_DEBUG group as ungrouped

Signed-off-by: Augusto Berndt <[email protected]>

Author: gudeh

src/gpl/src/placerBase.cpp

@@ -1015,12 +1015,24 @@ void PlacerBase::init()
       continue;
     }
 
-    if ((inst->dbInst() && inst->dbInst()->getGroup() != group_)
-        && inst->dbInst()->getGroup()->getType()
-               != odb::dbGroupType::VISUAL_DEBUG) {
+    odb::dbInst* db_inst = inst->dbInst();
+    if (!db_inst) {
       continue;
     }
 
+    odb::dbGroup* db_inst_group = db_inst->getGroup();
+    if (group_ == nullptr) {
+      if (db_inst_group
+          && db_inst_group->getType() != odb::dbGroupType::VISUAL_DEBUG) {
+        continue;
+      }
+    } else {
+      if (!db_inst_group || db_inst_group != group_
+          || db_inst_group->getType() == odb::dbGroupType::VISUAL_DEBUG) {
+        continue;
+      }
+    }
+
     if (inst->isFixed()) {
       // Check whether fixed instance is
       // within the corearea
@@ -1198,11 +1210,23 @@ void PlacerBase::initInstsForUnusableSites()
     if (!inst->isFixed()) {
       continue;
     }
-    if ((inst->dbInst() && inst->dbInst()->getGroup() != group_)
-        && inst->dbInst()->getGroup()->getType()
-               != odb::dbGroupType::VISUAL_DEBUG) {
+    odb::dbInst* db_inst = inst->dbInst();
+    if (!db_inst) {
       continue;
     }
+
+    odb::dbGroup* db_inst_group = db_inst->getGroup();
+    if (group_ == nullptr) {
+      if (db_inst_group
+          && db_inst_group->getType() != odb::dbGroupType::VISUAL_DEBUG) {
+        continue;
+      }
+    } else {
+      if (!db_inst_group || db_inst_group != group_
+          || db_inst_group->getType() == odb::dbGroupType::VISUAL_DEBUG) {
+        continue;
+      }
+    }
     std::pair<int, int> pairX = getMinMaxIdx(
         inst->lx(), inst->ux(), die_.coreLx(), siteSizeX_, 0, siteCountX);
     std::pair<int, int> pairY = getMinMaxIdx(