Thursday, 2019-05-16

*** tpb has joined #symbiflow00:00
*** jevinskie has joined #symbiflow02:55
*** jevinskie has quit IRC03:44
*** jevinskie has joined #symbiflow04:09
*** _whitelogger has quit IRC04:50
*** _whitelogger has joined #symbiflow04:52
*** filt3r has quit IRC05:26
*** Bertl_zZ is now known as Bertl_oO05:38
*** futarisIRCcloud has quit IRC06:10
*** jevinski_ has joined #symbiflow06:52
*** jevinskie has quit IRC06:53
*** futarisIRCcloud has joined #symbiflow07:16
*** OmniMancer has joined #symbiflow07:29
*** celadon_ has quit IRC09:14
*** celadon has joined #symbiflow09:19
*** kgugala has quit IRC09:28
*** tmichalak has quit IRC09:29
*** futarisIRCcloud has quit IRC09:30
*** celadon_ has joined #symbiflow09:44
*** celadon has quit IRC09:45
*** futarisIRCcloud has joined #symbiflow09:56
*** kgugala has joined #symbiflow10:27
*** tmichalak has joined #symbiflow10:27
sf-slack2<acomodi> mithro: Ok, I have pushed a simple script that makes the conversion both in the Symbiflow and VTR PRs and updated the `run_vtr_flow.pl` script to run the conversion11:52
*** futarisIRCcloud has quit IRC12:05
*** OmniMancer has quit IRC13:21
sf-slack2<mkurc> Good time of day14:17
sf-slack2<mkurc> @me1 @litghost The PR for Yosys which added support for parsing Verilog attributes on parameters has been merged to the YosysHQ/master.14:18
sf-slack2<mkurc> Shall I issue a PR from YosysHQ/master to Symbiflow/Yosys/master+wip to synchronize?14:18
sf-slack2<mkurc> Or make a new PR with the same changes to Symbiflow/Yosys/master+wip ?14:19
sf-slack2<mkurc> @litghost I also updated the document regarding v2x FASM support: https://docs.google.com/document/d/1J8lLT4SzlK_56-vBfz0f2DaNLHPn9rABUB0Fu1ljF4M14:23
tpbTitle: Google Docs - create and edit documents online, for free. (at docs.google.com)14:23
*** OmniMancer has joined #symbiflow14:34
litghostmkurc: PR from YosysHQ/master to Symbiflow/master+wip15:06
sf-slack2<acomodi> litghost, mithro: I have run less "massive" benchmarks to test mode selection in VTR. Interestingly the pack time is ~4% higher, but the whole vtr flow takes ~4% time less than master. https://github.com/verilog-to-routing/vtr-verilog-to-routing/pull/517#issuecomment-49302414215:41
tpbTitle: Mode selection feature by acomodi · Pull Request #517 · verilog-to-routing/vtr-verilog-to-routing · GitHub (at github.com)15:41
*** Ultrasauce has quit IRC15:41
sf-slack2<acomodi> litghost, mithro: moreover, I solved the issues on equivalent tiles after the upstream merge (a few things were modified by VTR devs regarding placement which affected my solution). Both the PRs (on symbiflow and VTR) should turn green15:47
litghostacomodi: Have the VTR devs responded to either PR yet?  Might be worth poking them again if they are green.16:06
litghostacomodi: > updated the `run_vtr_flow.pl` script to run the conversion16:08
litghostDid you add a test where run_vtr_flow runs on an arch.xml which already has tiles?16:08
sf-slack2<acomodi> litghost: good suggestion, I need to add to the script a check to see if the arch.xml is already compliant with the new format16:09
sf-slack2<acomodi> CI passed on VPR reg_tests, but failed on ODIN II ones. I need to apply the architecture conversion script also in there16:18
*** space_zealot has joined #symbiflow16:50
*** jevinski_ has quit IRC17:06
*** space_zealot has quit IRC17:07
*** jevinskie has joined #symbiflow17:08
*** space_zealot has joined #symbiflow17:08
*** Ultrasauce has joined #symbiflow17:25
*** space_zealot has quit IRC17:33
*** space_zealot has joined #symbiflow17:44
mithromkurc: Do you have the code / pull request for your attributes on parameters?18:01
sf-slack2<mkurc> @me1 Yes, I have the code. I didn't issue a PR. Currently it can store attributes of parameters within the RTLIL. But it can only output them to a JSON as the only backend I modified is JSON.18:19
sf-slack2<mkurc> Maybe it is a good idea to make the PR with it to Symbiflow/yosys/master+wip ?18:21
sf-slack2<mkurc> Once we agree that it has all functionalities we want we can then issue a PR to the YosysHQ18:22
litghostmkurc: Let's start the PR on master+wip and see if we can minimize impact to the RTLIL and yosys runtime.  If we can make a "zero-cost" PR for people not using parameter attributes, maybe that will be amenable18:29
sf-slack2<mkurc> Ok, then. I'll issue a WIP PR.18:30
sf-slack2<mkurc> @mithro @litghost There it is: https://github.com/SymbiFlow/yosys/pull/2518:45
tpbTitle: [WIP] Support for attributes on parameters in Verilog by mkurc-ant · Pull Request #25 · SymbiFlow/yosys · GitHub (at github.com)18:45
mithromkurc: Great!18:56
*** jevinskie has quit IRC19:47
hackerfooDoes anyone else use Emacs? If so, have you gotten it to indent Verilog nicely i.e not indenting to the open parenthesis?20:03
hackerfoohttps://stackoverflow.com/questions/30986217/modify-verilog-mode-indentation20:03
tpbTitle: emacs - Modify verilog mode indentation - Stack Overflow (at stackoverflow.com)20:03
*** proteusguy has quit IRC20:04
*** bjorkintosh has quit IRC20:38
*** OmniMancer has quit IRC21:02
*** Bertl_oO is now known as Bertl_zZ21:45
*** space_zealot has quit IRC22:20
*** space_zealot has joined #symbiflow23:48

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