◐ Shell
clean mode source ↗

bpo-24165: Add a freelist for long objects by methane · Pull Request #22884 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@methane

@methane methane commented

Oct 22, 2020

edited by bedevere-bot

Loading

Copy link Copy Markdown

Member

@scoder scoder left a comment

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

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

Looks good to me. A few comments need fixing.

Comment thread

Include/internal/pycore_interp.h Outdated

Comment thread

Objects/longobject.c Outdated

Co-authored-by: scoder <stefan_ml@behnel.de>

@pablogsal

Copy link Copy Markdown

Member

From the bpo issue:

Inada-san, you can run pyperformance with this workaround:
python -m pip install pyperformance==1.0.0
We are fixing the error soon after https://discuss.python.org/t/pep-641-using-an-underscore-in-the-version-portion-of-python-3-10-compatibility-tags/5513 lands

…65.zbVM5Q.rst

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>

@methane methane changed the title bpo-24165: Add freelist for long objects bpo-24165: Add a freelist for long objects

Oct 23, 2020

@methane methane closed this

Oct 24, 2020

@methane methane deleted the long-freelist branch

October 24, 2020 07:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scoder scoder scoder left review comments

@pablogsal pablogsal pablogsal left review comments

@vstinner vstinner Awaiting requested review from vstinner

@serhiy-storchaka serhiy-storchaka Awaiting requested review from serhiy-storchaka

Assignees

No one assigned

Labels

awaiting core review performance

Performance or resource usage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@methane @pablogsal @scoder @the-knights-who-say-ni @bedevere-bot