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 scope support to ControlFlowGraphVisitor #13

Closed
pjattke opened this issue Apr 28, 2020 · 1 comment
Closed

Add scope support to ControlFlowGraphVisitor #13

pjattke opened this issue Apr 28, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@pjattke
Copy link
Contributor

pjattke commented Apr 28, 2020

The ControlFlowGraphVisitor currently uses a variable's identifier only to distinguish between variables. This, however, does not uniquely identify a variable because there can be multiple variables with the same identifier, declared in different scopes. We therefore need to also take the scope into account.

This enhancement is already implemented in the CompileTimeExpressionSimplifier and uses scope information provided by the base Visitor class.

@AlexanderViand
Copy link
Collaborator

The new CFGV implementation has support for Scopes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants