Skip to content

Conversation

@rwy7
Copy link
Contributor

@rwy7 rwy7 commented Jul 16, 2025

We want to use the same macro for an inline layer across all compilation units (circuits) in a design. Remove the circuit name from the macro name.

This requires a change to the FIRRTL ABI, here:
chipsalliance/firrtl-spec#313

We want to use the same macro for an inline layer across all compilation units
(circuits) in a design.  Remove the circuit name from the macro name.
@rwy7 rwy7 requested review from darthscsi and seldridge as code owners July 16, 2025 15:19
@rwy7 rwy7 added the FIRRTL Involving the `firrtl` dialect label Jul 16, 2025
@rwy7 rwy7 merged commit 194a788 into llvm:main Jul 17, 2025
7 checks passed
@rwy7 rwy7 deleted the change-inline-layer-macro branch July 17, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FIRRTL Involving the `firrtl` dialect

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant