Skip to content

Conversation

@seldridge
Copy link
Member

Bump the version of serialized FIRRTL to version 5.1.0. This was
mistakenly not bumped when the version was updated in Chisel.

Signed-off-by: Schuyler Eldridge schuyler.eldridge@sifive.com

Bump the version of serialized FIRRTL to version 5.1.0.  This was
mistakenly not bumped when the version was updated in Chisel.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
@seldridge seldridge added the Bugfix Fixes a bug, will be included in release notes label Apr 9, 2025

// The version supported by the serializer.
val version = Version(4, 2, 0)
val version = Version(5, 1, 0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most code paths are probably going through the Chisel IR serializer. Hopefully we can delete this one soonish (part of the motivation for #4878)

@seldridge seldridge merged commit 669bca2 into main Apr 9, 2025
17 of 18 checks passed
@seldridge seldridge deleted the dev/seldridge/firrtl-serializer-bump-version branch April 9, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix Fixes a bug, will be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants