The-OpenROAD-Project / OpenSTA

OpenSTA engine
GNU General Public License v3.0
387 stars 167 forks source link

Liberty memory support #11

Closed mguthaus closed 5 years ago

mguthaus commented 5 years ago

The read_liberty command reads a cell library, but it does not recognize memories. These require support for the memory cell type. You can see an example from OpenRAM at:

https://github.com/VLSIDA/OpenRAM/blob/master/compiler/tests/golden/sram_2_16_1_scn3me_subm_TT_5p0V_25C.lib

Let me know if you have any questions or if you have suggestions for adjusting memory lib files for use in OpenSTA.

jjcherry56 commented 5 years ago

I fixed a stupid bug parsing the units in the referenced file but don't see anything that would prevent it from functioning correctly. The ports and timing groups are same as any other cell with busses. Is there some specific problem you are seeing?

mguthaus commented 5 years ago

My apologies! I was making a stupid mistake. It does seem to work now.