search

chipsalliance / firrtl-spec

The specification for the FIRRTL language
44 stars 28 forks source link

[patch] Allow layerblocks anywhere in a module. #216

Closed dtzSiFive closed 3 months ago

dtzSiFive commented 3 months ago

Support added for CIRCT here: https://github.com/llvm/circt/pull/7234 .

mmaloney-sf commented 3 months ago

For context, is this hinting at layer blocks under whens? Are there other situations we have our attention on as well?

dtzSiFive commented 3 months ago

For context, is this hinting at layer blocks under whens? Are there other situations we have our attention on as well?

Yep, that's what it's doing, sorry for not specifying that more clearly! Also, similarly, within match statements.

I don't know of other situations, attention or otherwise. LMK if have any concerns, please! 👍