Added cost_index_is_invalid() to check if RRIndexedDataId < 0

Signed-off-by: Ethan Rogers <[email protected]>

Author: ethanroj23

vpr/src/device/rr_graph_view.h

@@ -249,6 +249,11 @@ class RRGraphView {
         return node_storage_.node_cost_index(node);
     }
 
+    /** @brief Check if a node cost index is invalid (ie less than 0). This function is inlined for runtime optimization. */
+    bool cost_index_is_invalid(RRNodeId node) const {
+        return node_storage_.cost_index_is_invalid(node);
+    }
+
     /** @brief Return the fast look-up data structure for queries from client functions */
     const RRSpatialLookup& node_lookup() const {
         return node_lookup_;

vpr/src/route/check_rr_graph.cpp

@@ -327,7 +327,7 @@ void check_rr_node(int inode, enum e_route_type route_type, const DeviceContext&
                   "in check_rr_node: inode %d (type %d) had a ptc_num of %d.\n", inode, rr_type, ptc_num);
     }
 
-    if ((size_t)cost_index < 0 || (size_t)cost_index >= (int)device_ctx.rr_indexed_data.size()) {
+    if (rr_graph.cost_index_is_invalid(rr_node) || (size_t)cost_index >= (int)device_ctx.rr_indexed_data.size()) {
         VPR_FATAL_ERROR(VPR_ERROR_ROUTE,
                         "in check_rr_node: node %d cost index (%d) is out of range.\n", inode, cost_index);
     }

vpr/src/route/rr_graph_storage.h

@@ -189,6 +189,10 @@ class t_rr_graph_storage {
         return RRIndexedDataId(node_storage_[id].cost_index_);
     }
 
+    bool cost_index_is_invalid(RRNodeId id) const {
+        return node_storage_[id].cost_index_ < 0;
+    }
+
     Direction node_direction(RRNodeId id) const {
         return get_node_direction(
             vtr::array_view_id<RRNodeId, const t_rr_node_data>(