Add a new APIs reserve_nodes() and resize_nodes to RRGraphBuilder #1905
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR focuses on updating routing resource graph builder functions, where we use the refactored data structure
RRGraphBuilder
to shadow the discrete data structurerr_graph_storage
.This PR aims to fully refactored/deprecate the direct use of the legacy APIs
reserve()
andresize()
from therr_node
data structure if present there.After this PR, the
reserve_nodes()
andresize_nodes()
from the refactored data structureRRGraphBuilder
is the only way to use it.Checklist:
reserve()
andresize()
fromrr_node.cpp
andrr_node.h
(if present)reserve_nodes()
andresize_nodes()
to data structuresRRGraphBuilder
, whose comments are Doxygen compatiblereserve()
,resize()
withreserve_nodes()
,resize_nodes()
respectively, from respective client functions (if needed)Related Issue
This pull request is a follow-up PR on the routing resource graph refactoring effort #1805, #1868
Motivation and Context
This PR is a continuation of the refactoring effort with a focus on shadowing the
rr_graph_storage
APIs in theRRGraphBuilder
data structure.This PR refactored the
reserve_nodes()
andresize_nodes()
API among the other APIs in #1847, #1868How Has This Been Tested?
Types of changes
Checklist: