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

Refactor test helper related to logical driver change #739

Merged
merged 2 commits into from
Jul 5, 2023

Conversation

clavin-xlnx
Copy link
Member

No description provided.

Signed-off-by: Chris Lavin <[email protected]>
@clavin-xlnx clavin-xlnx added the needs new release Dependent on the next future release label Jul 5, 2023
@clavin-xlnx clavin-xlnx requested a review from eddieh-xlnx July 5, 2023 16:11
@clavin-xlnx clavin-xlnx merged commit e8d5532 into logical_driver Jul 5, 2023
@clavin-xlnx clavin-xlnx deleted the add_log_driver_flag branch July 5, 2023 16:20
clavin-xlnx added a commit that referenced this pull request Jul 5, 2023
* Fixes an issue with makeBlackBox trying to remove pins from renamed nets

Signed-off-by: Chris Lavin <[email protected]>

* Adding code to properly unroute site routing of a cell when removed

Signed-off-by: Chris Lavin <[email protected]>

* Tweak description

Signed-off-by: Chris Lavin <[email protected]>

* Add test, more fixes

Signed-off-by: Chris Lavin <[email protected]>

* Fix test failure

Signed-off-by: Chris Lavin <[email protected]>

* Update src/com/xilinx/rapidwright/design/DesignTools.java

Co-authored-by: eddieh-xlnx <[email protected]>
Signed-off-by: Chris Lavin <[email protected]>

* More fixes

Signed-off-by: Chris Lavin <[email protected]>

* Update dual-output route when unrouting correctly

Signed-off-by: Chris Lavin <[email protected]>

* Refactor test helper related to logical driver change (#739)

* Refactoring to shared test area

Signed-off-by: Chris Lavin <[email protected]>

* Add license header

Signed-off-by: Chris Lavin <[email protected]>

---------

Signed-off-by: Chris Lavin <[email protected]>

---------

Signed-off-by: Chris Lavin <[email protected]>
Co-authored-by: eddieh-xlnx <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs new release Dependent on the next future release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants