Parametrize plugin on Signal tycon

[bgamari]

clash-protocols must use CSignal, not Signal, for unidirectional signals in Circuits.

See clash-lang/clash-protocols#59.

[bgamari]

Moved into draft state as I would like to hear @martijnbastiaan's opinion on this.

[bgamari]

Ahh, it appears that ad3c557 and the preceding refactorings may have eliminated the need for this.

[martijnbastiaan]

My understanding is that this indeed handled correctly by the plugin, as of @jonfowler's PR. What's confusing here is that you can write Signal at any place you can write Fwd - even if the Fwd doesn't resolve to a Signal. That is to say, the example you post in clash-lang/clash-protocols#59 should probably switch to Fwd to make it less confusing. (And.. we should probably change clash-protocol's CSignal :-))

I wouldn't mind a second opinion from @jonfowler.

[martijnbastiaan]

I believe we've resolved the confusion that was part of this PR by now. @bgamari feel free to re-open if that's not the case.