[PhysNetlistReader] Warn and omit if PIP not found

Signed-off-by: Eddie Hung <[email protected]>
Xilinx · Jan 5, 2024 · f066980 · f066980
1 parent 80be5e0
commit f066980
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 6 deletions.
diff --git a/src/com/xilinx/rapidwright/interchange/PIPCache.java b/src/com/xilinx/rapidwright/interchange/PIPCache.java
@@ -101,6 +101,10 @@ public PIP getPIP(Tile tile, int wire0StringIdx, int wire1StringIdx) {
 
             return tile.getPIP(wire0Idx, wire1Idx);
         });
+
+        if (pip == null) {
+            return null;
+        }
         PIP newPIP = new PIP(pip);
         newPIP.setTile(tile);
         return newPIP;

diff --git a/src/com/xilinx/rapidwright/interchange/PhysNetlistReader.java b/src/com/xilinx/rapidwright/interchange/PhysNetlistReader.java
@@ -569,14 +569,22 @@ private void readRouteBranch(Set<Wire> stubWires,
                 }
 
                 PIP pip = getPIP(tile, pReader.getWire0(), pReader.getWire1());
-                pip.setIsPIPFixed(pReader.getIsFixed());
-                pip.setIsReversed(!pReader.getForward());
+                if (pip == null) {
+                    String wire0 = strings.get(pReader.getWire0());
+                    String wire1 = strings.get(pReader.getWire1());
+                    System.err.println("WARNING: PIP for tile " + strings.get(pReader.getTile()) +
+                            " from wire " + wire0 + " to wire " + wire1 + " not found; " +
+                            " omitting from net " + net.getName());
+                } else {
+                    pip.setIsPIPFixed(pReader.getIsFixed());
+                    pip.setIsReversed(!pReader.getForward());
 
-                if (stubWires.remove(pip.getEndWire())) {
-                    pip.setIsStub(true);
-                }
+                    if (stubWires.remove(pip.getEndWire())) {
+                        pip.setIsStub(true);
+                    }
 
-                net.addPIP(pip);
+                    net.addPIP(pip);
+                }
                 break;
             }
             case BEL_PIN:{