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

Google sync #1578

Merged
merged 9 commits into from
Feb 8, 2024
Merged

Google sync #1578

merged 9 commits into from
Feb 8, 2024

Conversation

rchen152
Copy link
Contributor

@rchen152 rchen152 commented Feb 8, 2024

No description provided.

martindemello and others added 9 commits February 7, 2024 16:46
I realized that the construct I was calling a "VM" is really more like a single
frame of a VM. (It looks awfully similar to the frame struct described here:
https://tech.blog.aknin.name/2010/07/22/pythons-innards-interpreter-stacks/.)
This also explains why I felt like my VM needed nested VMs =)

PiperOrigin-RevId: 604760281
- visitors.LookupExternalTypes: Now unwraps an alias if it finds one.
- visitors.CollextDependencies: Also check pytd.Modules for dependencies.

PiperOrigin-RevId: 605085587
* Adds a frame._Stack class for data stack management.
* Starts a basic abstract.py with BaseValue and PythonConstant classes.
* Fixes LOAD_CONST to look up constants from OrderedCode.consts. Changes
  FakeOrderedCode to allow setting consts.
* Writes a frame's final block's locals to a final_locals field on the frame.
* Implements STORE_NAME to exercise final_locals.
* Fixes a small typo (`=` instead of `:` for a type annotation) in blocks.py.

PiperOrigin-RevId: 605096370
This has been the case since Python 3.0.

PiperOrigin-RevId: 605122089
@rchen152 rchen152 merged commit 0dc0956 into main Feb 8, 2024
22 checks passed
@rchen152 rchen152 deleted the google_sync branch February 8, 2024 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants