FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. by markshannon · Pull Request #100753 · python/cpython · GitHub

/ cpython Public

GH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. - #100753

Merged
markshannon merged 2 commits into
python:mainfrom
faster-cpython:remove-load-attr-method-with-dict
Jan 5, 2023
Merged

GH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction.#100753
markshannon merged 2 commits into
python:mainfrom
faster-cpython:remove-load-attr-method-with-dict

Conversation

markshannon commented Jan 4, 2023
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Stats show that it doesn't work. It represents about 0.02% of the instructions executed and its miss rate is over 90%.

markshannon changed the title Remove LOAD_ATTR_METHOD_WITH_DICT instruction. Gh-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. Jan 4, 2023
markshannon changed the title Gh-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. GH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. Jan 4, 2023
markshannon merged commit f20c553 into python:main Jan 5, 2023
markshannon deleted the remove-load-attr-method-with-dict branch January 5, 2023 12:20
carljm added a commit to carljm/cpython that referenced this pull request Jan 5, 2023
* main:
  pythonGH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. (pythonGH-100753)
  pythonGH-100766: Note that locale.LC_MESSAGES is not universal (pythonGH-100702)
carljm added a commit to carljm/cpython that referenced this pull request Jan 5, 2023
* main:
  pythonGH-100288: Remove LOAD_ATTR_METHOD_WITH_DICT instruction. (pythonGH-100753)
  pythonGH-100766: Note that locale.LC_MESSAGES is not universal (pythonGH-100702)
  Drop myself from pathlib maintenance (python#100757)
  pythongh-100739: Respect mock spec when checking for unsafe prefixes (python#100740)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL