Skip to content

gh-145685: Update find_name_in_mro() to return a _PyStackRef (gh-145693)#145693

Merged
colesbury merged 1 commit intopython:mainfrom
colesbury:gh-145685-find-name-in-mro
Mar 10, 2026
Merged

gh-145685: Update find_name_in_mro() to return a _PyStackRef (gh-145693)#145693
colesbury merged 1 commit intopython:mainfrom
colesbury:gh-145685-find-name-in-mro

Conversation

@colesbury
Copy link
Contributor

@colesbury colesbury commented Mar 9, 2026

This is the first of a few PRs to fix the scaling bottleneck in _PyType_LookupStackRefAndVersion.

@colesbury
Copy link
Contributor Author

NEWS entry will be included in a follow-up when the scaling problem is fixed.

@colesbury colesbury marked this pull request as ready for review March 9, 2026 18:25
@colesbury colesbury requested a review from markshannon as a code owner March 9, 2026 18:25
@colesbury colesbury requested a review from mpage March 9, 2026 18:25
@colesbury colesbury changed the title gh-145685: Update find_name_in_mro() to return a _PyStackRef gh-145685: Update find_name_in_mro() to return a _PyStackRef (gh-145693) Mar 10, 2026
@colesbury colesbury merged commit f26eca7 into python:main Mar 10, 2026
98 of 100 checks passed
@miss-islington-app
Copy link

Thanks @colesbury for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 10, 2026
…ythongh-145693)

(cherry picked from commit f26eca7732ca9d0e6893e3fdfdfd4c25339c7155)

Co-authored-by: Sam Gross <colesbury@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Mar 10, 2026

GH-145769 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Mar 10, 2026
@colesbury colesbury deleted the gh-145685-find-name-in-mro branch March 10, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants