This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Greg Price
Recipients Greg Price, jdemeyer, sir-sigurd
Date 2019-08-24.23:22:38
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1566688958.43.0.896087473315.issue37837@roundup.psfhosted.org>
In-reply-to
Content
> Is there a particular reason to specifically call PyLong_FromSize_t? Seems like PyLong_FromLong is the natural default (and what we default to in the rest of the code), and it's what this ends up calling anyway.

Ah I see, the patch is meant to go on top of GH-15192, which makes PyLong_FromSize_t apply the small-int optimization itself.

As I just suggested on GH-15192, I'd like to see that PR apply the small-int optimization in the more broadly-used PyLong_FromUnsignedLong... and then I think the natural thing for this new function to do is to call that.

Still quite curious how LTO does, and also curious what compiler and flags you're using in benchmarks.
History
Date User Action Args
2019-08-24 23:22:38Greg Pricesetrecipients: + Greg Price, jdemeyer, sir-sigurd
2019-08-24 23:22:38Greg Pricesetmessageid: <1566688958.43.0.896087473315.issue37837@roundup.psfhosted.org>
2019-08-24 23:22:38Greg Pricelinkissue37837 messages
2019-08-24 23:22:38Greg Pricecreate