[ModuleInst] Fix placement behavior when requesting no overlaps (#1108)

* Fix no overlap behavior Signed-off-by: Chris Lavin <[email protected]> * Simplify code Signed-off-by: Chris Lavin <[email protected]> * Update src/com/xilinx/rapidwright/design/ModuleInst.java Signed-off-by: eddieh-xlnx <[email protected]> --------- Signed-off-by: Chris Lavin <[email protected]> Signed-off-by: eddieh-xlnx <[email protected]> Co-authored-by: eddieh-xlnx <[email protected]>
Xilinx · Nov 27, 2024 · 3836227 · 3836227
1 parent 0e13fbe
commit 3836227
Showing 1 changed file with 4 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;
 
@@ -356,6 +353,10 @@ public boolean place(Site newAnchorSite, boolean skipIncompatible, boolean allow
                         }
                     }
                 }
+                if (existingSiteInst != null) {
+                    unplace();
+                    return false;
+                }
             }
 
             if (newSite == null || existingSiteInst != null) {