Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@chiselbot
Copy link
Collaborator

[cd] Bump CIRCT from firtool-1.125.0 to firtool-1.126.0

This is an automated commit generated by the circt/update-circt GitHub
Action.

The following 3 commit(s) was/were automatically cherry-picked from the ci/ci-circt-nightly branch:

Release Notes

Bump CIRCT from firtool-1.125.0 to firtool-1.126.0.

Release notes for new CIRCT versions can be found at the following links:

seldridge and others added 4 commits July 25, 2025 19:02
Remove dropped OM Map features that will appear in CIRCT 1.126.0.  This
was removed in this patch [[1]].

[1]: llvm/circt@2beb8e7

Signed-off-by: Schuyler Eldridge <[email protected]>
The forthcoming FIRRTL 6.0.0 makes some changes to the ABI for inline
layers [[1]].  Specifically, the module name was dropped and the delimiter
_after_ "layer" was changed from '_' to '$'.  E.g., what was previously:

```
layer_Foo$A
```

Is now:

```
layer$A
```

Propagate this change into Chisel by updating the way that inline layers
self-report their ABI.  This is needed to keep Chisel compatible with
CIRCT 1.126.0.

[1]: chipsalliance/firrtl-spec@b88817b

Signed-off-by: Schuyler Eldridge <[email protected]>
Add an API to add `knownlayers` specifications to FIRRTL `extmodule`s.
This is a mechanism to declare that an external module was compiled with
support for specific layers.  This then allows a FIRRTL comopiler compiler
to know that it should automatically enable these layers if needed.

In order to make this work, a number of other changes had to be made.

First, move `atModuleBodyEnd` from `RawModule` to `BaseModule`.  This
allows capturing of the `Builder` state when the module closes.  Here,
this is used to capture the layers that are known to the `Builder` if a
module is later converted to a `Definition`.

Second, the default layers are now added to the `Builder` when it starts
as opposed to when it ends.  This allows for the default layers to be
captured by module or external modules and then emitted correctly if
needed (in the `Definition` or as FIRRTL `knownlayers`).

Third, when an `Instance` is created, any layers that it has need to be
added to the `Builder`.  This can arise if a `Definition` is imported via
an annotation and used to create a blackbox which will define additional
`knownlayers`.  If these layers are not added to the FIRRTL circuit, then
this is illegal FIRRTL.

Signed-off-by: Schuyler Eldridge <[email protected]>
This is an automated commit generated by the `circt/update-circt` GitHub
Action.
@chiselbot chiselbot added the Dependency Update Updates a dependency, will be included in release notes label Jul 25, 2025
@seldridge seldridge merged commit 7b57beb into main Jul 25, 2025
18 checks passed
@seldridge seldridge deleted the cd/update-circt-from-firtool-1.125.0-to-firtool-1.126.0 branch July 25, 2025 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dependency Update Updates a dependency, will be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants