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 zach.ware
Recipients brett.cannon, paul.moore, steve.dower, tim.golden, zach.ware
Date 2016-01-03.03:25:25
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1451791529.87.0.678319737086.issue25348@psf.upfronthosting.co.za>
In-reply-to
Content
Here's a patch that folds PGO into PCbuild\build.bat (removing PCbuild\build_pgo.bat) and defaults to using "python -m test --pgo" for training.  This should apply to all branches, but will need some slight adjustment for 2.7 (namely, s/-m test/-m test.regrtest/).  It might also be nice to leave a stub build_pgo.bat on 2.7 and 3.5 in case anyone actually uses it.

This patch also switches the default build environment to 'amd64' rather than 'x86_amd64' for -p x64 when it looks like it's running on a 64 bit OS.  PGO requires 'amd64' for 64 bit builds, and if x86_amd64 is available, 'amd64' should be as well.  'x86_amd64' is still used for non-PGO builds when -p x64 is specified and it looks like a 32 bit OS.  I considered switching the default platform target to x64 when running on 64 bit, but that's a separate issue.  It uses "%ProgramFiles(x86)"=="" to detect 32-bit, and gives a helpful message in the case of a false positive on that check.
History
Date User Action Args
2016-01-03 03:25:30zach.waresetrecipients: + zach.ware, brett.cannon, paul.moore, tim.golden, steve.dower
2016-01-03 03:25:29zach.waresetmessageid: <1451791529.87.0.678319737086.issue25348@psf.upfronthosting.co.za>
2016-01-03 03:25:29zach.warelinkissue25348 messages
2016-01-03 03:25:29zach.warecreate