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

Alias Analysis #7863

Closed
Tracked by #7685
farmaazon opened this issue Sep 21, 2023 · 0 comments · Fixed by #8189
Closed
Tracked by #7685

Alias Analysis #7863

farmaazon opened this issue Sep 21, 2023 · 0 comments · Fixed by #8189
Assignees
Labels
-gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints

Comments

@farmaazon
Copy link
Contributor

farmaazon commented Sep 21, 2023

Currently, we use some simplified algorithm for discovering where identifiers are defined and where used, which is then used to make a list of edges. This task is about creating a proper one, which will discover cases like one name being overshadowed by lambda arguments etc.

You can take inspiration from rust implementation, however that one is implemented on old AST (in the new AST the work should be theoretically easier).

@farmaazon farmaazon mentioned this issue Sep 21, 2023
@farmaazon farmaazon added d-intermediate Difficulty: some prior knowledge required p-high Should be completed in the next sprint -gui p-medium Should be completed in the next few sprints and removed p-high Should be completed in the next sprint labels Sep 21, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Sep 25, 2023
@farmaazon farmaazon assigned mwu-tow and unassigned kazcw Oct 13, 2023
@mwu-tow mwu-tow moved this from 📤 Backlog to ⚙️ Design in Issues Board Oct 16, 2023
@Frizi Frizi moved this from ⚙️ Design to 🔧 Implementation in Issues Board Oct 18, 2023
@mwu-tow mwu-tow moved this from 🔧 Implementation to 👁️ Code review in Issues Board Nov 5, 2023
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Nov 8, 2023
@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants