Skip to content
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

Changes to device level LVS #686

Closed

Conversation

d-m-bailey
Copy link
Contributor

  • Always include the STD_CELL_LIBRARY (previously defining LVS_EXTRA_STD_CELL_LIBRARY would cause STD_CELL_LIBRARY to be ignored)
  • Added LVS_EXTRA_SPICE variable (functionally equivalent to LVS_EXTRA_STD_CELL_LIBRARY but makes more sense for analog or sram modules.)
  • Flatten cells matching *_example_* during extract
  • Flatten small cells during extract

- Always include the STD_CELL_LIBRARY
- Added LVS_EXTRA_SPICE variable
- Flatten cells matching *_example_* during extract
- Flatten small cells during extract
@donn
Copy link
Collaborator

donn commented Nov 20, 2021

@Manarabdelaty Mind taking a look at this? It's a bit out of my depth.

@donn donn self-requested a review December 24, 2021 11:27
@donn
Copy link
Collaborator

donn commented Dec 24, 2021

@d-m-bailey I took the liberty of updating the changes- can you take a look? If they match your intention, I'll merge.

@d-m-bailey
Copy link
Contributor Author

@donn Actually, this program needs to be modified a bit more with the most recent version of lvs.tcl, so I'd recommend holding off on the merge for a couple days. I'm working on a new PR and I'll explain the issues in more detail there.

Most importantly, the addition of LVS_INSERT_POWER_PINS processing breaks the run_lvs_batch procedure in flow.tcl.

Also only the flattening of cells with names containing example is only a partial solution. See issue #796

@donn
Copy link
Collaborator

donn commented Dec 24, 2021

Gotcha. I'll wait.

@donn donn closed this Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants