pepijndevos changed the topic of #apicula to: Project Apicula: bitstream documentation and tooling for Gowin FPGAs https://github.com/YosysHQ/apicula -- logs https://freenode.irclog.whitequark.org/apicula
<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> true
<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??
<pepijndevos> https://github.com/YosysHQ/nextpnr/blob/master/gowin/arch.cc#L594 for wires there are duplicates from the way I add them.
<pepijndevos> https://github.com/YosysHQ/nextpnr/blob/master/gowin/arch.cc#L735 but pips should be unique... so yea weird bug. maybe some name getting mangled
<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