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.

Sign up for GitHub

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

Jump to bottom

decap cells should be marked "CORE SPACER" in lef #237

Closed
gadfort opened this issue Oct 27, 2024 · 3 comments
Closed

decap cells should be marked "CORE SPACER" in lef #237

gadfort opened this issue Oct 27, 2024 · 3 comments
Assignees
@KrzysztofHerman @b10346
Labels
enhancement New feature or request

Comments

@gadfort
Copy link
Contributor

gadfort commented Oct 27, 2024

Environment

OpenROAD uses this flag to determine if a cell can be removed during remove_fillers, without this the decaps cannot be removed for antenna fixes.

Expected Behavior

decaps marked as SPACER to allow them to properly interact with the tools

Actual Behavior

decaps are just marked CORE.
@KrzysztofHerman KrzysztofHerman self-assigned this Oct 28, 2024
@KrzysztofHerman KrzysztofHerman added the enhancement New feature or request label Oct 28, 2024
@b10346
Copy link

b10346 commented Nov 20, 2024

Ok, need to be investigated

b10346 added a commit to b10346/IHP-Open-PDK that referenced this issue Nov 25, 2024
@b10346
move back CORE to CORE SPACER for decaps, IHP-GmbH#237 issue
032c520 
Signed-off-by: Alexey Balashov <[email protected]>
@b10346 b10346 mentioned this issue Nov 25, 2024
Merged
2 tasks
@sergeiandreyev
Copy link
Contributor

Hi @gadfort, could you please confirm that the update is as expected on your side as well.

@gadfort
Copy link
Contributor Author

gadfort commented Nov 30, 2024

@sergeiandreyev yes, this works. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KrzysztofHerman KrzysztofHerman

@b10346 b10346

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@gadfort @sergeiandreyev @KrzysztofHerman @b10346