circt icon indicating copy to clipboard operation
circt copied to clipboard

[Seq] Add a new op interface for clock transformation ops. NFC

Open prithayan opened this issue 1 year ago • 1 comments

Add the clock gate, clock inverter and clock divider ops to a new clock transformation interface. This makes it easier to handle them and get the input clock directly.

prithayan avatar Jun 18 '24 16:06 prithayan

Does ClockMuxOp need to have this interface? In that case getClk returning a single value seems not composable.

Yeah, I was not sure about clock mux, for the use case that I was considering, ClockMuxOp needs to be handled separately, unlike the others . But it makes sense to include it as a clock transformation op, and let the client deal with it, if it returns multiple input clocks. I will update the interface to include mux ops.

prithayan avatar Jun 18 '24 18:06 prithayan

Closing, as this can be accomplished via a simple function, the itnerface is not relevant.

prithayan avatar Oct 29 '24 14:10 prithayan