SymbiFlow / vtr-verilog-to-routing

SymbiFlow WIP changes for Verilog to Routing -- Open Source CAD Flow for FPGA Research
https://verilogtorouting.org
Other
37 stars 12 forks source link

Branch: disable-check-route-option #74

Open acomodi opened 5 years ago

acomodi commented 5 years ago

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: 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

litghost commented 5 years ago

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