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

Add new RustIdent struct to track canonical-path and location info #903

Merged
merged 1 commit into from Feb 3, 2022

Conversation

philberty
Copy link
Member

This refactors our TyTy type abstractions to contain their repspective
canonical-path and location info. This cleans up alot of location tracking
for example when we have generic structures we create implicit hirids which
may or may not have location info this now tracks the location info of
the declaration of the type avoiding any confustion.

@philberty philberty added this to the Macro Expansion milestone Feb 2, 2022
@philberty philberty self-assigned this Feb 2, 2022
@philberty philberty added this to In progress in Control Flow 3 Macros via automation Feb 2, 2022
This refactors our TyTy type abstractions to contain their repspective
canonical-path and location info. This cleans up alot of location tracking
for example when we have generic structures we create implicit hirids which
may or may not have location info this now tracks the location info of
the declaration of the type avoiding any confustion.
@philberty
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Feb 3, 2022

Build succeeded:

@bors bors bot merged commit 3e6f9c2 into master Feb 3, 2022
Control Flow 3 Macros automation moved this from In progress to Done Feb 3, 2022
@philberty philberty deleted the phil/type-location-info branch February 3, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant