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
Fix iobuf inference #1349
Fix iobuf inference #1349
Conversation
hierarchy -check -auto-top | ||
|
||
# Start flow after library reading | ||
synth_xilinx -vpr -flatten -abc9 -nosrl -noclkbuf -nodsp -iopad -run prepare:check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this run from prepare
to check
while the other pathway runs everything?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you read the comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh - I think I understand now.
The first couple of lines in this section do the library reading. Then you start from the prepare stage (which is the first stage after library reading)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
if { $::env(USE_ROI) == "TRUE" } { | ||
synth_xilinx -vpr -flatten -abc9 -nosrl -noclkbuf -nodsp | ||
} else { | ||
# Read Yosys baseline library first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are these not needed for the other pathway?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean by "these"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 17+18.
… ROI-less targets. Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
748dd2d
to
8d4be9d
Compare
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
8d4be9d
to
567d5dc
Compare
Attempt to make #1339 pass CI.