Calyx Backend uses static groups instead of attribute

[calebmkim]

Simple change to use static group <lateny> instead of <"static"=latency> for the Calyx backend.

[rachitnigam]

I'm confused, I thought the latency attribute comes after the name? Also, can we get rid of the space between the name and the latency?

[calebmkim]

ah yeah my bad. I'll change it.

[rachitnigam]

One clarification question but LGTM otherwise!

[sampsyo]

Wahoo; nice work on this!

[rachitnigam]

@calebmkim just checking what's the status of this PR?

[calebmkim]

Ah, we can merge it. I was just thinking, though: without the control-promotion pass it will make very inefficient hardware: we will keep on instantiating a single fsm for each group. But maybe it'll be helpful just to check the correctness of our compile-static pass regardless?

[rachitnigam]

Hm, that's a fair point! Should we wait till control promotion stuff is done and then merge it as a part of our efforts to close out the static groups milestone?

[calebmkim]

Not going to merge this. We have decided to just emit @promotable as a hint that can be ignored.