<tpb>
Title: vlog_to_pbtype.xml tool using object *type* rather than name in output. · Issue #169 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com)
<daveshah>
mithro: afaik we discussed that back when, and chose that approach because it fitted in with the Artix and doing the "N" module business
<daveshah>
that way we just used an include
<daveshah>
it would be some restructuring to avoid that
<mithro>
daveshah: So, one way is to just add another layer of indirection
<daveshah>
mithro: yeah, that would probably work
<mithro>
daveshah: I'm unclear if your flattening the output in the JSON and how that interacts
<daveshah>
mithro: nope, shouldn't be flattening the output for pb_type generation, IIRC
<daveshah>
all modules should be in the JSON, so long as they are readable
<mithro>
daveshah: FYI -- you seem to do a lot of "if <condition>: assert False, <message>" -- you can just do a "assert not <condition>, <message>"
<mithro>
daveshah: Took me a while to figure out why your version felt strange :-P
[X-Scale] has joined #yosys
X-Scale has quit [Ping timeout: 276 seconds]
[X-Scale] is now known as X-Scale
seldridge has joined #yosys
mwk has quit [Ping timeout: 248 seconds]
mwk has joined #yosys
GuzTech has quit [Quit: Leaving]
AlexDaniel has quit [Ping timeout: 248 seconds]
proteus-guy has joined #yosys
fsasm has quit [Ping timeout: 240 seconds]
promach has quit [Quit: WeeChat 2.1]
leviathan has joined #yosys
leviathan has quit [Read error: Connection reset by peer]