Node.getAllWiresInNode() returns empty list when Vivado doesn't#1002
Draft
eddieh-xlnx wants to merge 1 commit intomasterfrom
Draft
Node.getAllWiresInNode() returns empty list when Vivado doesn't#1002eddieh-xlnx wants to merge 1 commit intomasterfrom
eddieh-xlnx wants to merge 1 commit intomasterfrom
Conversation
Signed-off-by: Eddie Hung <eddie.hung@amd.com>
Member
|
After some investigation, it appears that the node information is not present in our source information and even scouring across the device to get the alternate tile patterns, the information is simply not available. This will likely require a new spin of device files and will probably take some time to fix. |
Collaborator
Author
|
Interestingly, it seems that: also returns nothing, meaning that this isn't a terribly important node to begin with. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
returns 22 wires.
However, the testcase in this PR returns no wires.