Fix no overlap behavior

Signed-off-by: Chris Lavin <chris.lavin@amd.com>
Xilinx · Nov 19, 2024 · 0a4103d · 0a4103d
1 parent 9cd25a6
commit 0a4103d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/com/xilinx/rapidwright/design/ModuleInst.java b/src/com/xilinx/rapidwright/design/ModuleInst.java
@@ -39,9 +39,6 @@
 import com.xilinx.rapidwright.device.Site;
 import com.xilinx.rapidwright.device.SiteTypeEnum;
 import com.xilinx.rapidwright.device.Tile;
-import com.xilinx.rapidwright.edif.EDIFNet;
-import com.xilinx.rapidwright.edif.EDIFPortInst;
-import com.xilinx.rapidwright.edif.EDIFTools;
 import com.xilinx.rapidwright.util.MessageGenerator;
 import com.xilinx.rapidwright.util.Utils;
 
@@ -358,6 +355,11 @@ public boolean place(Site newAnchorSite, boolean skipIncompatible, boolean allow
                 }
             }
 
+            if (existingSiteInst != null && !allowOverlap) {
+                unplace();
+                return false;
+            }
+
             if (newSite == null || existingSiteInst != null) {
                 //MessageGenerator.briefError("ERROR: No matching site found." +
                 //    " (Template Site:"    + templateSite.getName() +