clifford changed the topic of #yosys to: Yosys Open SYnthesis Suite: http://www.clifford.at/yosys/ -- Channel Logs: https://irclog.whitequark.org/yosys
tpb has quit [Remote host closed the connection]
tpb has joined #yosys
<mwk> ohhh
<mwk> okay
<mwk> I see what's wrong..
<mwk> tux3: you can stop bisecting now, I understand the bug
<tux3> mwk, okay. thanks!
<tux3> I don't have a repro, but do you still want me to open an issue for it?
<mwk> *sigh* fuck verilog, seriously
<mwk> no, I'll take care of it
<tux3> thanks
<mwk> want a quick & dirty fix that probably leaves like 100 other instances of the same problem unfixed?
<tux3> sure, why not
<mwk> try out the mwk/fix-width-0 branch
<mwk> (beware: it's untested)
<tux3> oh wow, that commit is a little frightening. I did not know that was necessary ....
<mwk> it was not necessary before, because of a bug in verilog frontend
<tux3> ah
<mwk> here's a little problem
<mwk> you cannot have a 0-width wire in verilog
<mwk> which is quite fucking problematic, because you can in RTLIL
<tux3> :/
<mwk> and we're using verilog for converting one cell to another, and the cell you're converting can have 0-width ports, which is *not actually expressible* in Verilog
<mwk> so the input [A_WIDTH-1:0] A; line actually describes a 2-bit input
<mwk> (which happen to be A[-1] and A[0])
<mwk> by passing that down to $pos, which is an internal cell that is strongly validated and expects 0-width input, you get an error
<tux3> I see
<tux3> well, your branch seems to work. Or at least, yosys gives no errors
<tux3> thanks again
<mwk> alright
<mwk> I'll try to get it properly fixed tomorrow
<mwk> there are way more places like that
<mwk> and yeah, it is a recent regression, PR #2027 specifically
<tpb> Title: Fix 0-WIDTH wire handling in techmap rules. · Issue #2058 · YosysHQ/yosys · GitHub (at github.com)
<qu1j0t3> /b 14
BinaryLust has joined #yosys
emeb has quit [Quit: Leaving.]
_whitelogger has joined #yosys
citypw has joined #yosys
<ZipCPU> mwk tux3: Is there a zero-length wire something that I need to fix in the crossbar core that I might not know about? Was it the crossbar core causing the problem?
<mwk> no
<mwk> there is no way it could be your issue, because you cannot actually make a zero-length wire in verilog
<mwk> they appear because of yosys optimizations on RTLIL
<mwk> in this case, on an $alu input
<mwk> probably some const that got optimized away
Degi has quit [Ping timeout: 258 seconds]
Degi has joined #yosys
BinaryLust has quit [Ping timeout: 256 seconds]
DaKnig has quit [Ping timeout: 272 seconds]
DaKnig has joined #yosys
Asu has joined #yosys
jakobwenzel has joined #yosys
dys has joined #yosys
ayazar has joined #yosys
vidbina has joined #yosys
captain_morgan has quit [Quit: Ping timeout (120 seconds)]
captain_morgan has joined #yosys
futarisIRCcloud has quit [Quit: Connection closed for inactivity]
kraiskil has joined #yosys
kraiskil_ has joined #yosys
kraiskil has quit [Ping timeout: 265 seconds]
kraiskil_ has quit [Ping timeout: 260 seconds]
vidbina has quit [Ping timeout: 256 seconds]
kraiskil_ has joined #yosys
citypw has quit [Ping timeout: 240 seconds]
citypw has joined #yosys
kraiskil_ has quit [Read error: Connection reset by peer]
kraiskil_ has joined #yosys
vidbina has joined #yosys
kraiskil_ has quit [Read error: Connection reset by peer]
mkru has joined #yosys
kraiskil_ has joined #yosys
X-Scale` has joined #yosys
X-Scale has quit [Ping timeout: 272 seconds]
X-Scale` is now known as X-Scale
jakobwenzel has quit [Quit: jakobwenzel]
adjtm_ has quit [Remote host closed the connection]
adjtm has joined #yosys
kraiskil_ has quit [Read error: Connection reset by peer]
adjtm has quit [Remote host closed the connection]
adjtm has joined #yosys
adjtm has quit [Remote host closed the connection]
adjtm has joined #yosys
futarisIRCcloud has joined #yosys
kraiskil_ has joined #yosys
vidbina has quit [Ping timeout: 256 seconds]
vidbina has joined #yosys
X-Scale` has joined #yosys
X-Scale has quit [Ping timeout: 256 seconds]
X-Scale` is now known as X-Scale
kraiskil_ has quit [Ping timeout: 265 seconds]
jfcaron__ has joined #yosys
X-Scale` has joined #yosys
X-Scale has quit [Ping timeout: 246 seconds]
X-Scale` is now known as X-Scale
jfcaron__ is now known as jfcaron
citypw has quit [Ping timeout: 240 seconds]
vidbina has quit [Ping timeout: 256 seconds]
FFY00 has quit [Remote host closed the connection]
FFY00 has joined #yosys
FFY00 has quit [Read error: Connection reset by peer]
FFY00 has joined #yosys
jakobwenzel has joined #yosys
BinaryLust has joined #yosys
jakobwenzel has quit [Ping timeout: 272 seconds]
kraiskil_ has joined #yosys
FFY00 has quit [Remote host closed the connection]
FFY00 has joined #yosys
dys has quit [Ping timeout: 260 seconds]
kraiskil_ has quit [Ping timeout: 265 seconds]
kraiskil_ has joined #yosys
mkru has quit [Quit: Leaving]
jakobwenzel has joined #yosys
jakobwenzel has quit [Ping timeout: 256 seconds]
BinaryLust has quit [Ping timeout: 246 seconds]
dys has joined #yosys
ayazar has quit [Quit: ayazar]
FFY00 has quit [Read error: Connection reset by peer]
kraiskil_ has quit [Ping timeout: 264 seconds]
az0re has quit [Remote host closed the connection]
BinaryLust has joined #yosys
az0re has joined #yosys
BinaryLust has quit [Ping timeout: 265 seconds]
BinaryLust has joined #yosys
jakobwenzel has joined #yosys
jakobwenzel has quit [Client Quit]
<Forty-Bot> does yosys allow for some_module #(SOME_PARAMETER = foo) (/* inputs and outputs */) syntax?
<daveshah> Yes
<daveshah> If you mean in the module header?
<Forty-Bot> I mean when instantiating the module
<daveshah> No you need an instance name between the parameters and ports
<Forty-Bot> ah, ok
<Forty-Bot> thanks
<tpb> Title: picorv32/icebreaker.v at master · cliffordwolf/picorv32 · GitHub (at github.com)
ross_s has quit [Ping timeout: 260 seconds]
Asu has quit [Ping timeout: 256 seconds]
BinaryLust has quit [Ping timeout: 265 seconds]
BinaryLust has joined #yosys
az0re has quit [Remote host closed the connection]
az0re has joined #yosys
BinaryLust has quit [Ping timeout: 246 seconds]
jfcaron has quit [Ping timeout: 265 seconds]