◐ Shell
reader mode source ↗
Skip to content

bpo-44338: Port LOAD_GLOBAL to PEP 659 adaptive interpreter#26638

Merged
markshannon merged 11 commits into
python:mainfrom
faster-cpython:specialize-load-global
Jun 14, 2021
Merged

bpo-44338: Port LOAD_GLOBAL to PEP 659 adaptive interpreter#26638
markshannon merged 11 commits into
python:mainfrom
faster-cpython:specialize-load-global

Conversation

@markshannon

@markshannon markshannon commented Jun 10, 2021

Copy link
Copy Markdown
Member
  • Adds two specializations of LOAD_GLOBAL:

    • LOAD_GLOBAL_MODULE for module variables
    • LOAD_GLOBAL_BUILTIN for builtins.
  • Removes the old "opcache" mechanism.

  • Adds a few more stats (hidden behind a compile-time flag).

https://bugs.python.org/issue44338

@markshannon markshannon added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 10, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @markshannon for commit 8ad3e55 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 10, 2021

@ericsnowcurrently ericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

There are a few spots where things are a little unclear but mostly I was able to follow the change.

@markshannon markshannon merged commit eecbc7c into python:main Jun 14, 2021
jdevries3133 pushed a commit to jdevries3133/cpython that referenced this pull request Jun 19, 2021
…-26638)

* Add specializations of LOAD_GLOBAL.

* Add more stats.

* Remove old opcache; it is no longer used.

* Add NEWS
@markshannon markshannon deleted the specialize-load-global branch January 6, 2022 15:27
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.

4 participants