Matt Liberty
Matt Liberty
(btw, your image is cutoff)
Yes the error is meant to be fatal but it doesn't occur in the CI build. I am guessing you have some extraneous file with messages in it (like a...
try "grep -r 54 src/pdn" and see what files come up.
Do you have a screen shot of the PDN formerly generated?
> How so? It looks like they are: > > https://github.com/The-OpenROAD-Project/OpenROAD/blob/a2b9a98b5917fddaa9e99d801328f3f8d0e9d039/src/drt/src/io/io.cpp#L1521-L1528 Line 1528 is updating minWidth. io::Parser::initCutLayerWidth() updates width if unset from LEF
@rovinski I think it is something for another PR. I feel messing with the minWidth reflects a shortcoming in drt itself that should be fixed rather than hacking the constraints....
You'll have to fix the build errors but otherwise it looks ok
Is this a standard cell or a macro? We support flipping (not rotation) of standard cells in optimize_mirroring which is part of detailed placement.
Macro placement is separate from global placement. I don't think it support rotation or flipping but perhaps @mgwoo can comment.
Someone needs to start by analyzing the current results. Last I looked we got surprisingly little timing improvement when restructuring.