From d64ca2a1e6d3dc559ef46b4fdeba8186184459b9 Mon Sep 17 00:00:00 2001
From: Miodrag Milanovic <mmicko@gmail.com>
Date: Wed, 14 Jan 2026 15:10:32 +0100
Subject: [PATCH] print detected region and error if not found

---
 himbaechel/uarch/gatemate/ccf.cc     |  8 +++++-
 himbaechel/uarch/gatemate/gatemate.h |  1 +
 himbaechel/uarch/gatemate/pack.cc    | 43 +++++++++++++++++++++-------
 himbaechel/uarch/gatemate/pack.h     |  1 +
 4 files changed, 41 insertions(+), 12 deletions(-)

diff --git a/himbaechel/uarch/gatemate/ccf.cc b/himbaechel/uarch/gatemate/ccf.cc
index ea73a2a1..bc4bb16c 100644
--- a/himbaechel/uarch/gatemate/ccf.cc
+++ b/himbaechel/uarch/gatemate/ccf.cc
@@ -261,7 +261,13 @@ struct GateMateCCFReader
 
                                     IdString scopename(ctx, src_location.c_str());
                                     ctx->createRectangularRegion(scopename, x1, y1, x2, y2);
-                                    uarch->scopenames.emplace(scopename);
+                                    if (!uarch->scopenames.count(scopename))
+                                        log_error("Unknown scope name: %s in line %d\n", scopename.c_str(ctx), lineno);
+                                    else
+                                        log_info("    Constraining region '%s' to '%s'\n", scopename.c_str(ctx),
+                                                 pb_position.c_str());
+
+                                    uarch->scopenames_used.emplace(scopename);
                                 } else {
                                     log_error("Placebox format invalid: %s in line %d\n", pb_position.c_str(), lineno);
                                 }
diff --git a/himbaechel/uarch/gatemate/gatemate.h b/himbaechel/uarch/gatemate/gatemate.h
index 31aab310..54f403e5 100644
--- a/himbaechel/uarch/gatemate/gatemate.h
+++ b/himbaechel/uarch/gatemate/gatemate.h
@@ -110,6 +110,7 @@ struct GateMateImpl : HimbaechelAPI
     dict<int, IdString> index_to_die;
     dict<IdString, int> die_to_index;
     pool<IdString> scopenames;
+    pool<IdString> scopenames_used;
 
   private:
     bool getChildPlacement(const BaseClusterInfo *cluster, Loc root_loc,
diff --git a/himbaechel/uarch/gatemate/pack.cc b/himbaechel/uarch/gatemate/pack.cc
index fc7e39e0..45a32fc4 100644
--- a/himbaechel/uarch/gatemate/pack.cc
+++ b/himbaechel/uarch/gatemate/pack.cc
@@ -514,21 +514,41 @@ void GateMatePacker::assign_clocks()
     }
 }
 
+static IdString get_scopename(Context *ctx, CellInfo &ci)
+{
+    std::string scope = "top";
+    if (ci.attrs.count(ctx->id("scopename"))) {
+        scope = str_or_default(ci.attrs, ctx->id("scopename"), "");
+        scope = "top " + scope;
+    } else if (ci.attrs.count(ctx->id("hdlname"))) {
+        scope = str_or_default(ci.attrs, ctx->id("hdlname"), "");
+        scope = "top " + scope;
+    }
+    return IdString(ctx, scope.c_str());
+}
+
+void GateMatePacker::find_regions()
+{
+    for (auto &cell : ctx->cells) {
+        CellInfo &ci = *cell.second;
+        IdString name = get_scopename(ctx, ci);
+        uarch->scopenames.emplace(name);
+    }
+    if (uarch->scopenames.size() > 1) {
+        log_info("Detected regions..\n");
+        for (auto &scope : uarch->scopenames) {
+            log_info("    %s\n", scope.c_str(ctx));
+        }
+    }
+}
+
 void GateMatePacker::assign_regions()
 {
     log_info("Assign cell region based on attributes..\n");
     for (auto &cell : ctx->cells) {
         CellInfo &ci = *cell.second;
-        std::string scope = "top";
-        if (ci.attrs.count(ctx->id("scopename"))) {
-            scope = str_or_default(ci.attrs, ctx->id("scopename"), "");
-            scope = "top " + scope;
-        } else if (ci.attrs.count(ctx->id("hdlname"))) {
-            scope = str_or_default(ci.attrs, ctx->id("hdlname"), "");
-            scope = "top " + scope;
-        }
-        IdString name = IdString(ctx, scope.c_str());
-        if (uarch->scopenames.count(name))
+        IdString name = get_scopename(ctx, ci);
+        if (uarch->scopenames_used.count(name))
             ctx->constrainCellToRegion(ci.name, name);
         if (ci.attrs.count(id_GATEMATE_DIE) != 0) {
             std::string die_name = str_or_default(ci.attrs, id_GATEMATE_DIE, "");
@@ -579,6 +599,8 @@ void GateMatePacker::fix_regions()
 void GateMateImpl::pack()
 {
     const ArchArgs &args = ctx->args;
+    GateMatePacker packer(ctx, this);
+    packer.find_regions();
     if (args.options.count("ccf")) {
         parse_ccf(args.options["ccf"].as<std::string>());
     }
@@ -612,7 +634,6 @@ void GateMateImpl::pack()
     if (strategy == MultiDieStrategy::REUSE_CLK1 || strategy == MultiDieStrategy::FULL_USE)
         preferred_die = 0;
 
-    GateMatePacker packer(ctx, this);
     if (forced_die == IdString())
         packer.assign_regions();
     packer.pack_constants();
diff --git a/himbaechel/uarch/gatemate/pack.h b/himbaechel/uarch/gatemate/pack.h
index 533fa7bb..dbf24801 100644
--- a/himbaechel/uarch/gatemate/pack.h
+++ b/himbaechel/uarch/gatemate/pack.h
@@ -72,6 +72,7 @@ struct GateMatePacker
     void reassign_clocks();
     void copy_clocks();
     void assign_clocks();
+    void find_regions();
     void assign_regions();
     void fix_regions();