Fix FIXME: restore DesignTools.createPossiblePinsToStaticNets();

Xilinx · Nov 24, 2022 · 4dfecb0 · 4dfecb0
1 parent e2cc447
commit 4dfecb0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/com/xilinx/rapidwright/rwroute/PartialRouter.java b/src/com/xilinx/rapidwright/rwroute/PartialRouter.java
@@ -101,8 +101,7 @@ public PartialRouter(Design design, RWRouteConfig config, Collection<SitePinInst
     protected void preprocess() {
         // Pre-processing of the design regarding physical net names pins
         DesignTools.makePhysNetNamesConsistent(design);
-        // FIXME: Enable for PartialRouter
-        // DesignTools.createPossiblePinsToStaticNets(design);
+        DesignTools.createPossiblePinsToStaticNets(design);
         DesignTools.createMissingSitePinInsts(design);
     }
 

diff --git a/test/src/com/xilinx/rapidwright/rwroute/TestRWRoute.java b/test/src/com/xilinx/rapidwright/rwroute/TestRWRoute.java
@@ -52,7 +52,7 @@ private static void assertAllSinksRouted(List<SitePinInst> pins) {
 
     private static void assertAllSinksRouted(Design design) {
         for (Net net : design.getNets()) {
-            if (net.getSource() == null) {
+            if (net.getSource() == null && !net.isStaticNet()) {
                 // Source-less nets may exist in out-of-context design
                 continue;
             }
@@ -128,12 +128,11 @@ public void testNonTimingDrivenPartialRouting() {
         String dcpPath = RapidWrightDCP.getString("picoblaze_partial.dcp");
         Design design = Design.readCheckpoint(dcpPath);
         DesignTools.createMissingSitePinInsts(design);
-        // FIXME:
-        // DesignTools.createPossiblePinsToStaticNets(design);
+        DesignTools.createPossiblePinsToStaticNets(design);
 
         List<SitePinInst> pinsToRoute = new ArrayList<>();
         for (Net net : design.getNets()) {
-            if (net.getSource() == null) {
+            if (net.getSource() == null && !net.isStaticNet()) {
                 // Source-less nets may exist since this is an out-of-context design
                 continue;
             }