Skip to content

Use METH_FASTCALL where it makes sense#681

Merged
asvetlov merged 12 commits intomasterfrom
fastcall
Jan 22, 2022
Merged

Use METH_FASTCALL where it makes sense#681
asvetlov merged 12 commits intomasterfrom
fastcall

Conversation

@asvetlov
Copy link
Copy Markdown
Member

@asvetlov asvetlov commented Jan 22, 2022

After the change, MultiDict.add() is 2.2 times faster than METH_VARARGS | METH_KEYWORDS version.
CIMultiDict shows not such an exciting boost, it is only 1.5 times faster. Not bad anyway.

@asvetlov asvetlov requested a review from iemelyanov as a code owner January 22, 2022 19:01
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Jan 22, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2022

Codecov Report

Merging #681 (51f0200) into master (1aa1deb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #681   +/-   ##
=======================================
  Coverage   93.65%   93.65%           
=======================================
  Files           5        5           
  Lines         504      504           
=======================================
  Hits          472      472           
  Misses         32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1aa1deb...51f0200. Read the comment docs.

@asvetlov asvetlov merged commit 3da0f79 into master Jan 22, 2022
@asvetlov asvetlov deleted the fastcall branch January 22, 2022 20:40
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 5, 2024
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 16, 2024
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 16, 2024
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 16, 2024
This reverts commit 3da0f79.
Partially, at least. Under Python 3.13+ only.
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 16, 2024
This reverts commit 3da0f79.
Partially, at least. Under Python 3.13+ only.
webknjaz added a commit to webknjaz/multidict that referenced this pull request Jan 16, 2024
This reverts commit 3da0f79.
Partially, at least. Under Python 3.13+ only.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant