Skip to content

Conversation

@openroad-robot
Copy link
Contributor

Replaced by Hierarchical RTL-MPL (mpl2; to be renamed mpl)

Replaced by Hierarchical RTL-MPL (mpl2; to be renamed mpl)

Signed-off-by: Matt Liberty <[email protected]>
@gadfort
Copy link
Collaborator

gadfort commented Oct 29, 2024

@maliberty can you hold off on merging this until I've verified this won't cause a whole host of issues in my CI?

@maliberty
Copy link
Member

I was planning to wait a bit for feedback. I do want to drop it as we no longer do any development or support for mpl.

@gadfort
Copy link
Collaborator

gadfort commented Oct 29, 2024

Minor metrics update for tinyRocket_nangate45 for clock skew

Signed-off-by: Matt Liberty <[email protected]>
@AcKoucher
Copy link
Contributor

AcKoucher commented Jan 22, 2025

@gadfort Would it be possible to give your CI another go with the changes here now that #5809 was merged?

@gadfort
Copy link
Collaborator

gadfort commented Jan 22, 2025

@AcKoucher sure, but I would need this PR to be updated to include those changes.

@maliberty
Copy link
Member

@gadfort merged

@gadfort
Copy link
Collaborator

gadfort commented Jan 23, 2025

@AcKoucher all the ones that are active pass

@maliberty
Copy link
Member

@AcKoucher all the ones that are active pass

Does that mean this is ok to merge from your perspective?

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@maliberty maliberty merged commit 44e3614 into The-OpenROAD-Project:master Jan 25, 2025
11 checks passed
@maliberty maliberty deleted the rm-old-mpl branch January 25, 2025 05:47
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.

4 participants