-
Notifications
You must be signed in to change notification settings - Fork 117
LEF obstruction for nwell seems incomplete #300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Or maybe it's omitted on purpose because it's far away from the borders ? |
I'm not sure why it's missing the nwell shape. I don't see a prBoundary layer; if it doesn't exist, then magic will take the extents of geometry as the bounding box, and should include everything inside that bounding box in the LEF file. |
There is a prBoundary, it definitely englobes all the shapes shown. This is the design btw : https://github.com/TinyTapeout/tt_asw_1v8/blob/main/mag/tt_asw_1v8.mag |
Thanks for the link to the layout. That's clearly wrong, and I'll need to investigate. |
@smunaut : I found out that "lef write" has an inconsistent policy about MASTERSLICE layers. The nwell that got found was part of the routine checking for stuff outside the prBoundary, which includes MASTERSLICE layers. But the routine that fills in the center area when using the "-hide" option does not include MASTERSLICE layers. After thinking it over, I decided to allow the center area to be filled with nwell. That might be a bad solution, though, because it implies that there is nwell in areas of the cell where there isn't any. So I have provisionally implemented that solution, but I am still thinking about it. |
So this is originally from a .mag
If you compare the generated GDS:

And then the generated LEF:

One of the large N-Well obstruction is missing from the LEF.
(Those all comes from PCell subcells in the original MAG. On the top it's two PCell overlapping their guardring).
The text was updated successfully, but these errors were encountered: