-
Notifications
You must be signed in to change notification settings - Fork 16
Hijacking barstools for tapeout passes #3
Comments
@chick do you need any of the files currently in the repo? Looks like this was all experimentation you did that's now been integrated w/ Chisel? |
@shunshou You are correct, I do not need anything that is in this repo. |
Thanks for creating the issue. I started working on some of this and hope to have it done over the weekend. |
@colinschmidt I started a branch called tapeout with basic scala style / build.sbt + Adam's ResetInverter thing. If you don't already have one, maybe I should just PR and merge? |
Let's just merge your branch. I have the scala style directory and build.sbt but the pfpmp contents, so I can just base my changes on yours. |
Ok, I PRed. One thing I briefly asked Adam is -- with the new transform methodology, it's not evident which order the passes/transformed are run (since you're not adding them manually via your custom compiler). He thinks (and can correct me if I'm wrong) that it's just based off of which annotation is seen first. In case passes/transforms might depend on each other, there needs to be a mechanism to state pass priority. (i.e. inferrw must happen before replseqmem). At least among the passes I mentioned in this issue, there's prob no need, but just food for thought. |
Yeah thats a good point. The GenerateTop and GenerateHarness need to be ordered to achieve the desired affect. I guess we'll need another firrtl feature request @azidar :) |
We will need to track chipsalliance/firrtl#446 for ordering firrtl passes via annotations. |
@azidar @colinschmidt and I talked about using barstools for writing tapeout-related Firrtl passes.
Here are a list of passes that exist in other locations that should eventually be moved here:
Future stuff:
The text was updated successfully, but these errors were encountered: