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

[BOT] Update openroad_app #601

Merged

Conversation

openlane-bot
Copy link
Collaborator

This is an automated PR.

See the individual commits for details.

@openlane-bot openlane-bot force-pushed the openroad_app-update-branch branch 8 times, most recently from a26fbe3 to 3e85915 Compare September 17, 2021 00:35
openroad_app -> f7a44b69256c0b2a1dce50362997c4091d7f6a06
@openlane-bot openlane-bot force-pushed the openroad_app-update-branch branch from 3e85915 to 5189f57 Compare September 17, 2021 12:17
@donn
Copy link
Collaborator

donn commented Sep 17, 2021

Okay, this is way too late in the merge window for a major change like this but, this may have some memory leak fixes and some larger designs are getting murdered.

@donn donn self-requested a review September 17, 2021 17:18
@donn donn merged commit d7f041b into The-OpenROAD-Project:master Sep 17, 2021
donn added a commit to donn/openlane that referenced this pull request Sep 19, 2021
- The-OpenROAD-Project#607 - Lack of legalization was causing routing issues with OpenROAD.

- The-OpenROAD-Project#601 - There is a cryptic "maze ripup wrong" message that makes aes_cipher fail: see The-OpenROAD-Project/OpenROAD#1110
@donn donn mentioned this pull request Sep 19, 2021
donn added a commit that referenced this pull request Sep 19, 2021
- #607 - Lack of legalization was causing routing issues with OpenROAD.

- #601 - There is a cryptic "maze ripup wrong" message that makes aes_cipher fail: see The-OpenROAD-Project/OpenROAD#1110

- CU of aes_cipher reduced to 40 (50 was causing issues with TR)

- Fixed or_issue
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