◐ Shell
reader mode source ↗
Skip to content

bpo-46406: Faster single digit int division.#30626

Merged
mdickinson merged 4 commits into
python:mainfrom
gpshead:long-faster-divide
Jan 23, 2022
Merged

bpo-46406: Faster single digit int division.#30626
mdickinson merged 4 commits into
python:mainfrom
gpshead:long-faster-divide

Conversation

@gpshead

@gpshead gpshead commented Jan 16, 2022

Copy link
Copy Markdown
Member

This expresses the algorithm in a more basic manner resulting in better
instruction generation by todays compilers.

See https://mail.python.org/archives/list/python-dev@python.org/thread/ZICIMX5VFCX4IOFH5NUPVHCUJCQ4Q7QM/#NEUNFZU3TQU4CPTYZNF3WCN7DOJBBTK5

https://bugs.python.org/issue46406

This expresses the algorithm in a more basic manner resulting in better
instruction generation by todays compilers.

See https://mail.python.org/archives/list/python-dev@python.org/thread/ZICIMX5VFCX4IOFH5NUPVHCUJCQ4Q7QM/#NEUNFZU3TQU4CPTYZNF3WCN7DOJBBTK5
@gpshead gpshead marked this pull request as ready for review January 17, 2022 03:15
@gpshead gpshead added the performance Performance or resource usage label Jan 17, 2022

@mdickinson mdickinson 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

LGTM, modulo a cast needed to silence the compiler on Windows.

gpshead and others added 2 commits January 22, 2022 10:37
Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
@mdickinson

Copy link
Copy Markdown
Member

Still LGTM. Thank you!

@mdickinson mdickinson merged commit c7f20f1 into python:main Jan 23, 2022
@bedevere-bot

Copy link
Copy Markdown

@mdickinson: Please replace # with GH- in the commit message next time. Thanks!

@gpshead gpshead deleted the long-faster-divide branch January 23, 2022 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Performance or resource usage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants