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][Elaboration] Support labels #8107

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Conversation

maerhart
Copy link
Member

No description provided.

@maerhart maerhart added the RTG Involving the `rtg` dialect label Jan 22, 2025
@maerhart maerhart requested review from youngar and darthscsi January 22, 2025 15:28
@@ -1026,6 +1109,7 @@ class Elaborator : public RTGOpVisitor<Elaborator, FailureOr<DeletionKind>> {
DenseMap<Value, ElaboratorValue> state;

uint64_t virtualRegisterID = 0;
uint64_t uniqueLabelID = 1;
Copy link
Member Author

Choose a reason for hiding this comment

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

Move to shared state.

Copy link
Member

Choose a reason for hiding this comment

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

Similar to VirtualRegisters, UniqueLabels could rely on a malloc'd address instead giving each one a unique ID.

@maerhart maerhart force-pushed the maerhart-rtg-emission-labels branch from 3989308 to 135f067 Compare January 31, 2025 19:16
Base automatically changed from maerhart-rtg-emission-labels to main January 31, 2025 19:35
@maerhart maerhart force-pushed the maerhart-rtg-elaboration-labels branch from 98b2f4c to 122c8bc Compare January 31, 2025 19:39
@maerhart maerhart force-pushed the maerhart-rtg-elaboration-labels branch from 122c8bc to 1eb5b5f Compare January 31, 2025 19:57
@maerhart maerhart merged commit b73d2d7 into main Jan 31, 2025
5 checks passed
@maerhart maerhart deleted the maerhart-rtg-elaboration-labels branch January 31, 2025 19:58
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