vtr-verilog-to-routing icon indicating copy to clipboard operation
vtr-verilog-to-routing copied to clipboard

Branch: disable-check-route-option

Open acomodi opened this issue 6 years ago • 1 comments

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?
  • [x] is this currently part of the Conda package?
  • [ ] is this properly cleaned up in our local repositories?

Tracker / branch / PR & other useful links

Issue with more info: https://github.com/verilog-to-routing/vtr-verilog-to-routing/issues/537 code diff: https://github.com/verilog-to-routing/vtr-verilog-to-routing/compare/master...SymbiFlow:disable-check-route-option

acomodi avatar Jun 07 '19 09:06 acomodi

This one can be removed when check-route isn't so slow.

litghost avatar Jun 07 '19 15:06 litghost