Wednesday, 2019-03-27

*** tpb has joined #vtr-dev00:00
mithrokem_: https://github.com/kmurray/libblifparse/pull/2 <- Code formatting starting with the deps repositories that you own16:16
tpbTitle: Add format target for source code formatting by kgugala · Pull Request #2 · kmurray/libblifparse · GitHub (at github.com)16:16
litghostkem_: I think I have an example graph where the router fails to find a valid path because it doesn't add all valid fan outs to the heap.  It sounds vaguely like what https://github.com/verilog-to-routing/vtr-verilog-to-routing/commit/46661d08602a71ed641a91e67fa2833cc1aefb62#diff-4ac57f609972e5b875fd99f90d6f518e was supposed to addressed, but it is occurring on something close to master22:04
tpbTitle: vpr: Improve robustness of high-fanout routing on some architectures · verilog-to-routing/[email protected] · GitHub (at github.com)22:04
litghostkem_: Might be a poor interaction between non-configurable channel connections?22:05
litghostkem_: I think the bug is here https://github.com/verilog-to-routing/vtr-verilog-to-routing/blob/master/vpr/src/route/route_timing.cpp#L185522:08
tpbTitle: vtr-verilog-to-routing/route_timing.cpp at master · verilog-to-routing/vtr-verilog-to-routing · GitHub (at github.com)22:08
litghostkem_: Basically during non-configurable edge expansion, the heap gets a downstream node from the source, but in a case where there is a high-fanout non-configurable net, it is possible that the wrong node is added to the heap22:09
litghostkem_: Until a valid route is found, all edges need to be expanded, yes?  But that doesn't appear to be happening22:09
litghostkem_: Non-configurable seems to be a red herring22:29
litghostkem_: Ah, found the issue, https://github.com/verilog-to-routing/vtr-verilog-to-routing/blob/master/vpr/src/route/route_timing.cpp#L1751 is filtering the route22:41
tpbTitle: vtr-verilog-to-routing/route_timing.cpp at master · verilog-to-routing/vtr-verilog-to-routing · GitHub (at github.com)22:41
mithrolitghost: https://vtr-verilog-to-routing.readthedocs.io/en/latest/vpr/command_line_usage.html#cmdoption-vpr-const-gen-inference23:11
tpbTitle: Command-line Options Verilog-to-Routing 8.0.0-dev documentation (at vtr-verilog-to-routing.readthedocs.io)23:11
mithrolitghost: btw do you have VTR_ENABLE_DEBUG_LOGGING on?23:20
litghostmithro: Ya23:20

Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!