Граф коммитов

6 Коммитов

Автор SHA1 Сообщение Дата
sebmarchand aa821bdef5 Fix typo in merge_pgc_files.py
BUG=528632
TBR=scottmg@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2933493002
Cr-Original-Commit-Position: refs/heads/master@{#478182}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: f38c70f4d88bc524ca299a5692510852a5d307be
2017-06-09 03:30:11 +00:00
Sebastien Marchand f34c461c7a Add a retry logic in merge_pgc_files.py
pgomgr.exe sometime fails to merge too many files at the same time, with
what seems to be a memory issue (process peak at ~4GB before failing,
it's a 64-bit process so it's probably a size_t/uint32_t issue). I've a
local repro that I'll send to MS, but in the meantime this fix is
required to fix the official build (and the PGO ones in general).

Bug: 729894
Change-Id: I9c862c0179de50ee89b22a03789c9fb0a1071d48
Reviewed-on: https://chromium-review.googlesource.com/528354
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#478066}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 0c6adbef4d5f4722e2a224c90164f30ec9cf26dd
2017-06-08 20:10:47 +00:00
sebmarchand 1caf3a69f3 Update merge_pgc_files.py in preparation for VS2017
pgomgr.exe has been moved to a different location in VS2017 so the logic
that find it should be updated.

It can't just be copied to the build directory because we always run the
x64 bit version of this binary (and we don't want to mix x86 and x64
binaries in the same build dir)

BUG=719319

Review-Url: https://codereview.chromium.org/2884613003
Cr-Original-Commit-Position: refs/heads/master@{#473438}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: ab3a1821bb57a51d11be2b038b3a698236ccd5fc
2017-05-20 15:00:06 +00:00
sebmarchand 1e7adaaf94 Add a script to merge the PGC files.
Workaround for a flakyness in pgomgr.exe where it can run out of address
space while trying to merge all the PGC files at the same time.

This will also allow to upload the pgd file before running the optimize step.

BUG=674956

Review-Url: https://codereview.chromium.org/2575543002
Review-Url: https://codereview.chromium.org/2575543002
Cr-Original-Commit-Position: refs/heads/master@{#449082}
Committed: 6c18b1e861
Cr-Original-Original-Commit-Position: refs/heads/master@{#448332}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 4a19948e1bc13344a301c439445d36b44c207bb1
2017-02-08 20:37:22 +00:00
sebmarchand a5b79d9869 Revert of Add a script to merge the PGC files (patchset #6 id:120001 of https://codereview.chromium.org/2575543002/ )
Reason for revert:
Breaks the PGO bots.

Original issue's description:
> Add a script to merge the PGC files.
>
> Workaround for a flakyness in pgomgr.exe where it can run out of address
> space while trying to merge all the PGC files at the same time.
>
> This will also allow to upload the pgd file before running the optimize step.
>
> BUG=674956
>
> Review-Url: https://codereview.chromium.org/2575543002
> Cr-Commit-Position: refs/heads/master@{#448332}
> Committed: 6c18b1e861

TBR=scottmg@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=674956

Review-Url: https://codereview.chromium.org/2680683002
Cr-Original-Commit-Position: refs/heads/master@{#448561}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 6c2f8b4bd17f67d03b3e4d85401a0eaa68597afa
2017-02-07 06:11:01 +00:00
sebmarchand df35027309 Add a script to merge the PGC files.
Workaround for a flakyness in pgomgr.exe where it can run out of address
space while trying to merge all the PGC files at the same time.

This will also allow to upload the pgd file before running the optimize step.

BUG=674956

Review-Url: https://codereview.chromium.org/2575543002
Cr-Original-Commit-Position: refs/heads/master@{#448332}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 6c18b1e861ec6a9c96d24b3bd962688e50c83597
2017-02-06 19:05:05 +00:00