<
pepijndevos>
wow what? A gowin-nextpnr regression? weird
<
pepijndevos>
ughhhh probably the same changes that were applied to nextpnr generic will also have to be applied to gowin
<
pepijndevos>
I don't really understand what this change is about tbh
<
trabucayre>
I search :-/
<
trabucayre>
generic and gowin have more or less same changes
<
trabucayre>
Yep need to read all commits ...
<
pepijndevos>
maybe comment with you regression on that issue
<
trabucayre>
but not now -> day job :-/
<
pepijndevos>
Unless you or litghost fix it I guess I'll have to debug it later
<
trabucayre>
maybe need to add a note about which commit to use
<
pepijndevos>
I'll make a comment
<
pepijndevos>
Just parroting your info and to get discussion started/receive updates/remember
<
trabucayre>
it's seems to have one pip twice
<
gatecat>
is there any easy way to test this?
<
gatecat>
ok I can reproduce it with --test, let's see
<
trabucayre>
pepijndevos: thanks to add this comment for me :)
<
pepijndevos>
wait it's pips, not wires??
<
gatecat>
yeah seems like it
<
trabucayre>
pepijndevos: yep addPip fails
<
trabucayre>
two pip.src_id seems same
<
trabucayre>
(If I've understood )
FabM has quit [Remote host closed the connection]
<
trabucayre>
gatecat: whoua! it's fast!
<
pepijndevos>
I'll give it a review tomorrow hopefully