Skip to content

Add string concatenation support#5241

Merged
seldridge merged 1 commit intomainfrom
dev/seldridge/string_concat
Mar 20, 2026
Merged

Add string concatenation support#5241
seldridge merged 1 commit intomainfrom
dev/seldridge/string_concat

Conversation

@seldridge
Copy link
Copy Markdown
Member

Add a new API for creating variadic string_concat FIRRTL operations.
This is useful for working with string properties and specifically for
setting the name of domains which are derived from other domains.

AI-assisted-by: Augment (Sonnet 4.5)

Release Notes

Add new Property.concat operation that can be used to (currently) just
concatenate strings. This is useful for building up properties from smaller
strings. This maps to the string_concat FIRRTL operation.

@seldridge seldridge added the Feature New feature, will be included in release notes label Mar 20, 2026
@seldridge seldridge requested a review from jackkoenig March 20, 2026 16:43
Base automatically changed from dev/seldridge/use-builder-exceptions-in-properties to main March 20, 2026 22:00
Add a new API for creating variadic `string_concat` FIRRTL operations.
This is useful for working with string properties and specifically for
setting the name of domains which are derived from other domains.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
@seldridge seldridge force-pushed the dev/seldridge/string_concat branch from 95d3a15 to 503f39b Compare March 20, 2026 22:05
@seldridge seldridge marked this pull request as ready for review March 20, 2026 22:05
@seldridge seldridge enabled auto-merge (squash) March 20, 2026 22:12
@seldridge seldridge merged commit 21c0323 into main Mar 20, 2026
27 checks passed
@seldridge seldridge deleted the dev/seldridge/string_concat branch March 20, 2026 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature, will be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants