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.
Chipyard bumped to a newer version of the toolchain (FireMarshal bump that also removed the need for
e2fsprogs
in the conda env) that needs to be updated in the conda requirements. I also pinneds3fs
to=0.4.6
to avoid issues whereaiobotocore
(a prereq ofs3fs
) would be updated too far and break pytests.Additionally, we plan on bumping to a newer version of Verilator that removes the C++ top-level in favor of Rocket ChipsThis is broken in FireSim so I will delay this to the following release.TestDriver.v
as the top level. This is compatible with current Verilator makefiles.Related PRs / Issues
UI / API Impact
Verilog / AGFI Compatibility
No changes to Verilog so no AGFIs were regenerated.
Contributor Checklist
changelog:<topic>
label?ci:fpga-deploy
label?Please Backport
label?Reviewer Checklist (only modified by reviewer)
Note: to run CI on PRs from forks, comment
@Mergifyio copy main
and manage the change from the new PR.changelog:<topic>
label?