New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
speedup some comparisons #47356
Comments
This patch is an experiment in making faster some of the most common Test minimum run-time average run-time |
Raymond, would you want to take a look? |
You may get better timings if you more the types-are-equal test inside Do something like: In general, I'm not too keen on adding this kind of dispatch code to |
Hello,
I get no discernable difference.
Some minimal type testing is necessary if we want to implement the I'm attaching a patch which reduces the additional dispatch to a Test minimum run-time average run-time |
Here is a new patch without any dispatch shortcut in ceval.c, just Test minimum run-time average run-time The patch seems fairly uncontroversial to me, I'll commit it soon if |
If there's not a hurry, would like to review this a bit more when I get |
No pb! |
On 2008-12-13 16:08, Antoine Pitrou wrote:
Why have you removed the complete error handling section in
|
Le lundi 15 décembre 2008 à 14:41 +0000, Marc-Andre Lemburg a écrit :
Because the only error that can occur is a TypeError when one of the two |
On 2008-12-15 16:34, Antoine Pitrou wrote:
Ah, sorry, just saw that this is just for Py3. The fast-path would probably also make sense for Py2 (keeping the |
I committed the patch, which will also help bpo-1717. Thanks! |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: