Improve logging for the yosys toolchain #285
Merged
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.
The yosys toolchain was mostly slapped together as proof of concept. After using it in anger for a bit I've found some missing functionality around the logs we're exposing so this exposes more of the correct logs, like we'd like to get warnings from the ghdl synthesizer, the yosys log and the next pnr log with area and timing.
The ghdl thing is a bit strange in that it logs warnings to stderr when run as a plugin in yosys but the warnings don't make it into the log file, probably due to how yosys is logging things. I've captured the stderr there and written it to a file, and printed it to stderr so that when we get a non-zero exit you see the errors in the terminal.