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

Parse __ptr[T] and __slice[T] #4522

Merged
merged 6 commits into from
May 7, 2023
Merged

Parse __ptr[T] and __slice[T] #4522

merged 6 commits into from
May 7, 2023

Conversation

AlicanC
Copy link
Contributor

@AlicanC AlicanC commented May 2, 2023

Just parsing for typed pointers and slices. Went with long and distinct names for now but we could change them to something like &T and &[T] in the future.

The only tricky part was this, where I had to fix how we parse impl items: https://github.com/FuelLabs/sway/pull/4522/files#diff-51ff5479fa74960c097173172a99b15be1dbd4d858960e619eca1ae740816552

@AlicanC AlicanC self-assigned this May 2, 2023
@AlicanC AlicanC added the compiler: frontend Everything to do with type checking, control flow analysis, and everything between parsing and IRgen label May 2, 2023
@AlicanC AlicanC marked this pull request as ready for review May 2, 2023 19:26
@AlicanC AlicanC requested a review from a team May 2, 2023 19:26
@AlicanC AlicanC enabled auto-merge (squash) May 2, 2023 19:40
@IGI-111 IGI-111 requested a review from a team May 2, 2023 20:45
@IGI-111 IGI-111 added breaking May cause existing user code to break. Requires a minor or major release. compiler: parser Everything to do with the parser labels May 2, 2023
@AlicanC AlicanC requested a review from IGI-111 May 3, 2023 15:27
@AlicanC AlicanC merged commit 657a10b into master May 7, 2023
@AlicanC AlicanC deleted the jc/typed-ptr-parsing branch May 7, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking May cause existing user code to break. Requires a minor or major release. compiler: frontend Everything to do with type checking, control flow analysis, and everything between parsing and IRgen compiler: parser Everything to do with the parser
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants