Peter

Results 85 comments of Peter

> @arlpetergadfort comments? The grid looks okay when spot checking. The missing straps were removed because they didn't connect to anything so their presence is not need. I see connections...

@maliberty yes, I think so. With -secondary_net options. It would be good if the OP could verify it works for them, but there are test cases that address this in...

@maliberty I'm unable to look at the files until late today. But if they are using the OpenLane flow, I don't know if there is a way to specify multiple...

In that case, the current setup will not be able to connect them. OL will need to be augmented to support multiple different definitions of macros (or the macro will...

@dineshannayya I found a bug in the instance grid code, see PR: https://github.com/The-OpenROAD-Project/OpenROAD/pull/1997 For the PLL, because it has metal on met5, PDNGEN will not place straps over it, so...

@dineshannayya this should allow you to connect the macros: https://github.com/The-OpenROAD-Project/OpenROAD/pull/2134 It will require placement to ensure the straps can go through the macro (the placement uploaded doesn't).

> There is no OBS blockage in met4 or met5 (so the behavior is going to be the same). If you move it over by 5um in X I think...

My list: - CTS, the tree built something makes fixing setup/hold worse by poorly selecting buffers, improving CTS would help with that I think. - Resizer setup repair gives up...

> @arlpetergadfort is the minimum gap to deal with the lack of 1 site filler in some pdks or something else? @maliberty yes, exactly. Without it increasing density would become...

Current ODB just generates an error when attempting to load new DEFs (https://github.com/The-OpenROAD-Project/OpenROAD/blob/f8c243bd63829123a58a9c22112c8d76b681522a/src/odb/src/defin/definReader.cpp#L1630 this is in the form of a printf instead of going to the logger). So that would...