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

Consider symbol table + scopes for name binding #38

Closed
mighdoll opened this issue Nov 12, 2024 · 1 comment
Closed

Consider symbol table + scopes for name binding #38

mighdoll opened this issue Nov 12, 2024 · 1 comment
Assignees
Milestone

Comments

@mighdoll
Copy link
Contributor

The linker needs to match symbol references to symbol declarations. Currently it does this with a combination of resolveImport() and localRef().

Consider unifying the reference to declaration bindings with a scoped symbol tree approach, like esbuild describes. It might be cleaner, it'd likely be faster, and it'd more upgradable to future submodules/namespaces..

@mighdoll mighdoll moved this to Todo in wesl-js Nov 12, 2024
@mighdoll mighdoll added this to wesl-js Nov 12, 2024
@mighdoll mighdoll added this to the M1 milestone Nov 18, 2024
@mighdoll mighdoll self-assigned this Nov 18, 2024
@mighdoll
Copy link
Contributor Author

mighdoll commented Jan 9, 2025

completed!

@mighdoll mighdoll closed this as completed Jan 9, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in wesl-js Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

1 participant