Early rail analysis #67
Conversation
| }""") | ||
|
|
||
| if not corners: | ||
| # TODO: remove hardcoded my_view string |
There was a problem hiding this comment.
What's the alternative here?
There was a problem hiding this comment.
There isn't, for now. This is just default as generated from hammer, but it could be a property.
| @@ -829,6 +920,18 @@ def run_innovus(self) -> bool: | |||
| # Create open_chip script pointing to latest (symlinked to post_<last ran step>). | |||
| with open(self.open_chip_tcl, "w") as f: | |||
| f.write("read_db latest") | |||
| if self.ran_early_rail: | |||
There was a problem hiding this comment.
Would it be better to just write out a separate "open_early_rail" script in the same directory instead of combining it with open_chip?
There was a problem hiding this comment.
I don't think so, since it requires reading the database anyway? I guess an open_early_rail script would do the same as existing open_chip but with the following added.
| @@ -33,3 +33,7 @@ par.innovus: | |||
| # Note that this requires an optional licence (enccco). | |||
| # type: bool | |||
| use_cco: true | |||
|
|
|||
| # Estimated power for early rail analysis. | |||
There was a problem hiding this comment.
A nit, but it might be a bit more clear to say "Estimated total power used by early rail analysis in Watts." "This is a designer input estimate for the supply..."
Implements early rail analysis after
place_opt_design. Key differences to rail analysis in the Voltus plugin is:par.rail_analysis_power(see Early rail analysis hammer#611) over all the placed instances, given the power routing. In my opinion, this is better than running static power analysis since it allows the user to do a coarse what-if analysis using a different power number.open_chipscript loads newest rail analysis database (in the case of multiple MMMC corners, this will be the last corner) and prompts the user for whether they want to display the results in the GUI. If so, this is what shows up using theTinyRocketConfigexample from the Chipyard ASAP7 example (obviously, the results don't make sense since the SRAMs aren't characterized):