• Paolo Severini's avatar
    [Test] Make CSuite benchmark runner work better on Windows · a974dd7e
    Paolo Severini authored
    The csuite.py script does not work correctly on Windows. It runs
    correctly in baseline mode, but there are two problems when running in
    compare mode:
    
    1. In compare mode the output of benchmark.py is piped to the
       compare-baseline.py script, but Windows only execute python files if
       python.exe is the default program to open '.py' files, and this is
       not the case, by default, when python is installed as part of the
       depot_tools.
    
       Fix: explicitly add the 'python' command before compare-baseline.py.
    
    2. By default CSuite prints the results to stdout using escapes codes
       that add color highlights. But this does not work on Windows when
       compare-baseline.py is launched with a pipe:
    
       python test/benchmarks/csuite/benchmark.py <...> |
           python test/benchmarks/csuite/compare-baseline.py <baseline_results>
    
       Fix: Do not use a pipe. Write the benchmark numbers for the
       compare-run into a separate file, and pass the path to this file to
       compare-baseline.py
    
    Change-Id: Ic22d5bd4b47901f0ba0f35bc2496441346d21c6a
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2656855Reviewed-by: 's avatarMichael Stanton <mvstanton@chromium.org>
    Commit-Queue: Paolo Severini <paolosev@microsoft.com>
    Cr-Commit-Position: refs/heads/master@{#72807}
    a974dd7e
Name
Last commit
Last update
.github Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.flake8 Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
RISCV_OWNERS Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...