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.

Sign up for GitHub

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

Jump to bottom

Add NameResolver #267

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add NameResolver #267

wants to merge 5 commits into from
+54 −52

Conversation

mtshiba
Copy link
Member

@mtshiba mtshiba commented Nov 29, 2022

Implement a name-resolver.

This allows for the separation of name resolution functionality from Context, as well as constant reordering and variable circularity checks.

TODOs:

  • implement basic name-resolving
    • check referencing before definition
    • unused variable warning
    • deleted variable check
  • implement constant-reordering
  • circularity check

@mtshiba mtshiba changed the title Add name_resolve.rs Add NameResolver Nov 29, 2022
@mtshiba mtshiba self-assigned this Nov 29, 2022
@mtshiba mtshiba force-pushed the main branch 2 times, most recently from 67d834a to 68a27f8 Compare June 2, 2023 14:46
@mtshiba mtshiba force-pushed the main branch 5 times, most recently from 49c4ae6 to 02a97b0 Compare August 30, 2023 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mtshiba mtshiba

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mtshiba @C-BJ