Rachit Nigam

Results 504 comments of Rachit Nigam
trafficstars

Marking this as stale. @ethanuppal feel free to remove and add thoughts if on how we can get this merged without the nightly features.

Okay, then let's trigger the remaining tests and see if there are any failures. You need to mark the PR ready for review when you're ready.

@ekiwi wanted another pair of eyes on this in case you think there could be any problems changing the `always_comb` to `always @*`. I'm going to fix the failing tests...

Closing in favor of https://github.com/calyxir/calyx/pull/2330.

@ethanuppal can you finalize the changes and answer the questions so we can merge this PR?

We generally dump out the VCD for the design and query specific signals we care about. However, you are right: this is a pretty baroque approach to doing things and...

Do you have a specific design in mind for something like this?

I think there is a further opportunity to simplify things by merging the `MonoSig` and `MonoDeferred` structs. Since #310, a component is immediately monomorphized when we come across its instantiation...

@sampsyo I was looking at the code and I actually think the current separation is fine. `tools/yxi` generates the interface to a Calyx program while `yxi/` is really about generating...