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 steve.dower
Recipients Mark.Shannon, neonene, paul.moore, rhettinger, steve.dower, tim.golden, vstinner, zach.ware
Date 2021-09-07.21:57:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1631051827.01.0.327723924982.issue45116@roundup.psfhosted.org>
In-reply-to
Content
> I'm not sure if PGO builds are reproducible, since there is a training step which requires to run a non-deterministic workload (the Python test suite which is somehow randomized by I/O timings and other stuffs).

I'm 99% sure it's a tracing PGO rather than sampling, so provided everything runs in the same order and follows the same codepaths, wall-clock timings shouldn't have a significant effect.

Without looking at the generated code, it may be more effective to try and force the functions called from the macro to never be inlined. The optimiser may be missing that the calls are uncommon, and trying to inline them first, then deciding that the whole merged function is too big to inline in places where it would be useful.

There's also no particular reason we need to use these tests as the profile, it's just that nobody has taken the time to come up with anything better. I'd rather see us invest time in generating a good profile rather than trying to hand-optimise inlining. (Though the test suite is good in many ways, because it makes sure all the extension modules are covered. We definitely want as close to full code coverage as we can get, at least for happy paths, but may need more eval loop in the profile if that's what needs help.)
History
Date User Action Args
2021-09-07 21:57:07steve.dowersetrecipients: + steve.dower, rhettinger, paul.moore, vstinner, tim.golden, Mark.Shannon, zach.ware, neonene
2021-09-07 21:57:07steve.dowersetmessageid: <1631051827.01.0.327723924982.issue45116@roundup.psfhosted.org>
2021-09-07 21:57:07steve.dowerlinkissue45116 messages
2021-09-07 21:57:06steve.dowercreate