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

Statically eliminate VFFSL #81

Closed
asottile opened this issue May 23, 2014 · 4 comments
Closed

Statically eliminate VFFSL #81

asottile opened this issue May 23, 2014 · 4 comments
Labels
feature improves-performance needs-new-parser

Comments

@asottile
Copy link
Collaborator

We should be able to know the following things at compile time:

  1. The list of globals
  2. The list of parameters
  3. The list of locals

Which should allow us to make a normal python lookup in a lot of cases instead of a cheetah variable (assuming we have autokey and autocall off)

Blocked by #80 because this would involve considering things before searchList, which is currently not the behaviour
@asottile
Copy link
Collaborator Author

After #174 VFN will not be used any more in the optimized case

@asottile
Copy link
Collaborator Author

asottile commented Jul 7, 2015 •
edited

VFN is no longer used. #241, #189, and #200 are necessary to call this fixed

@asottile asottile mentioned this issue Jun 7, 2016
Merged
@asottile
Copy link
Collaborator Author

This is basically unblocked (other than removing the enable_auto_self setting)

@asottile
Copy link
Collaborator Author

Via #245

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature improves-performance needs-new-parser
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@asottile