8000 bpo-46020: Optimize long_pow for the common case by tim-one · Pull Request #30555 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

bpo-46020: Optimize long_pow for the common case #30555

New issue 8000

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 12, 2022
Merged

bpo-46020: Optimize long_pow for the common case #30555

merged 1 commit into from
Jan 12, 2022

Conversation

tim-one
Copy link
Member
@tim-one tim-one commented Jan 12, 2022 •
edited by bedevere-bot
Loading

This cuts a bit of overhead by not initializing the table of small
odd powers unless it's needed for a large exponent.

https://bugs.python.org/issue46020

@tim-one
bpo-46020: Optimize long_pow for the common case
cff11fb 8000 
This cuts a bit of overhead by not initializing the table of small
odd powers unless it's needed for a large exponent.
@tim-one tim-one self-assigned this Jan 12, 2022
@tim-one tim-one added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 12, 2022
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @tim-one for commit cff11fb 🤖

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 Jan 12, 2022
@tim-one tim-one marked this pull request as draft January 12, 2022 05:05
@tim-one tim-one marked this pull request as ready for review January 12, 2022 05:07
@tim-one tim-one merged commit fc05e6b into python:main Jan 12, 2022
@tim-one tim-one deleted the longpow branch January 12, 2022 18:55
@rhettinger rhettinger mentioned this pull request Apr 10, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tim-one tim-one

Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tim-one @bedevere-bot @the-knights-who-say-ni
0