-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add symbiflow-vtr-optimized with PGO enabled #143
Conversation
( | ||
curl $SYMBIFLOW_INSTALL_URL | tar xJ -C symbiflow | ||
) & | ||
FETCH_SYMBIFLOW_PID=$! |
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.
This task runs in parallel with the first build, so that downloading and unpacking the SymbiFlow install package doesn't take any additional time.
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.
The eblif should be either sourced from another repo, or compressed.
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.
Comments in
Signed-off-by: Dusty DeWeese <[email protected]>
The eblif is compressed now (180KB). |
The |
I've added an issue to update this after verilog-to-routing/vtr-verilog-to-routing#1564 is merged: #155 |
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.
Couple things.
stdout is too long. Consider piping to a file and greping for error/warnings, etc? Or just redirect to a file and output something when an error occurs?
CI is red:
The job exceeded the maximum time limit for jobs, and has been terminated.
This may be because of the stdout length?
GitHub Actions has significantly more powerful hardware and a much longer timeout, so I think we might want to wait until after that is landed so we can take advantage of that? |
This PR is superseded by hdl/conda-eda#60 |
This builds VPR, and then collects profile data, and uses that to compile VPR again with profile-guided optimization.