Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RTG] Support partial sequence substitutions #8148

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

maerhart
Copy link
Member

No description provided.

@maerhart maerhart added the RTG Involving the `rtg` dialect label Jan 30, 2025
@maerhart maerhart requested review from youngar and darthscsi January 30, 2025 11:51
@maerhart maerhart force-pushed the maerhart-rtg-randomize-sequence-op branch from 03cdb45 to fafb31a Compare February 3, 2025 08:52
@maerhart maerhart force-pushed the maerhart-rtg-partial-sequence-substitutions branch from 761f04f to 4849514 Compare February 3, 2025 08:55
Copy link
Contributor

@fabianschuiki fabianschuiki left a comment

Choose a reason for hiding this comment

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

This is a neat way of materializing a symbol as an SSA value. I wonder what other uses this pattern might have 🤔

@maerhart
Copy link
Member Author

maerhart commented Feb 4, 2025

One use-case is the context_switch op where you have to pass a sequence with 3 arguments of specific types. If multiple switches have a similar sequence you can create one sequence that has > 3 arguments and partially substitute the first N-3 to specialize for one particular switch instead of copy pasting the whole sequence.

@maerhart maerhart force-pushed the maerhart-rtg-randomize-sequence-op branch from fafb31a to c0ba88c Compare February 4, 2025 16:35
Base automatically changed from maerhart-rtg-randomize-sequence-op to main February 4, 2025 16:35
@maerhart maerhart force-pushed the maerhart-rtg-partial-sequence-substitutions branch from 4849514 to 5b33227 Compare February 4, 2025 16:36
@maerhart maerhart merged commit c572294 into main Feb 4, 2025
3 checks passed
@maerhart maerhart deleted the maerhart-rtg-partial-sequence-substitutions branch February 4, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RTG Involving the `rtg` dialect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants