Skip to content

Conversation

edwardcwang
Copy link
Member

No description provided.

Copy link
Contributor

@colinschmidt colinschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good. I think the clock constraints stuff is pretty similar between cadence and synopsys, so we might be able to share more code than usual (although I don't think we've thought about how to organize truly standard things) . We'll also eventually need internally generated clocks but this is a great start.

clocks = self.get_clock_ports()
for clock in clocks:
# TODO: are units in Synopsys always in ns?
output.append("create_clock {0} -name {0} -period {1}".format(clock.name, clock.period.value_in_units("ns")))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't this need a [get_ports {0}] or something?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently not, the first {0} captures the port name...

@edwardcwang edwardcwang merged commit 1490603 into master Nov 8, 2017
@edwardcwang edwardcwang deleted the clocks branch November 8, 2017 19:05
harrisonliew added a commit that referenced this pull request Mar 20, 2023
harrisonliew added a commit that referenced this pull request Mar 20, 2023
* merge synopsys and cadence plugins into hammer

* get e2e tests working with asap7 on a machines

* add ICV link to docs

* incorporate genus retiming commit

* incorporate genus ILM blackboxes commit

* fix type error

* add defines for sky130 post-syn sim

* Update keys for sky130 at BWRC used for #740

* -top should be optional (#34)

* extraneous type:ignores with local mypy

* Revert "extraneous type:ignores with local mypy"

This reverts commit b0f5f95.

* openroad at bwrc moved to common location

---------

Co-authored-by: Harrison Liew <harrisonliew@gmail.com>
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