You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Why did we need this? (what does this change enable us to do)
This is a temporary performance enhancement that allows to disable the check_route routine in several VPR steps (routing and genfasm).
The reasons behind the slow check routine run-time are still unknown, and it seems to affect only SymbiFlow tests.
What did it change?
Disables the check route routine in the vpr_api.cpp file.
Should it be merged upstream - if not, when can we delete it?
It should not be merged upstream. We can delete it as soon as the performance of check route are back to normal and do not slow down the whole VPR flow process.
What is needed to get this merged / deleted?
We need to explore the reasons behind the slow run-time of the check_route routine as trivially we cannot permanently disable it.
is the implementation work to make suitable for merging / deletion completed?
Is there an associated test?
is this currently part of the Conda package?
is this properly cleaned up in our local repositories?
Why did we need this? (what does this change enable us to do)
This is a temporary performance enhancement that allows to disable the check_route routine in several VPR steps (routing and genfasm).
The reasons behind the slow check routine run-time are still unknown, and it seems to affect only SymbiFlow tests.
What did it change?
Disables the check route routine in the
vpr_api.cpp
file.Should it be merged upstream - if not, when can we delete it?
It should not be merged upstream. We can delete it as soon as the performance of check route are back to normal and do not slow down the whole VPR flow process.
What is needed to get this merged / deleted?
We need to explore the reasons behind the slow run-time of the check_route routine as trivially we cannot permanently disable it.
Tracker / branch / PR & other useful links
Issue with more info: verilog-to-routing#537
code diff: verilog-to-routing/vtr-verilog-to-routing@master...SymbiFlow:disable-check-route-option
The text was updated successfully, but these errors were encountered: