diff options
| author | Daniel Podder <dapodd@microsoft.com> | 2017-07-10 11:37:25 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-07-10 11:37:25 -0700 |
| commit | aa63b76ac1c64b78f20edff35708477f3afac713 (patch) | |
| tree | 5cf8cafc3ac54ca8ce88a52a0db099386ec89f86 /tests | |
| parent | 28258cea0d6a7cfce257c9142a149d2d69f88efc (diff) | |
| download | coreclr-aa63b76ac1c64b78f20edff35708477f3afac713.tar.gz coreclr-aa63b76ac1c64b78f20edff35708477f3afac713.tar.bz2 coreclr-aa63b76ac1c64b78f20edff35708477f3afac713.zip | |
Fix optdata bootstrap test script & Linux skiprestoreoptdata behavior (#12541)
Fixes #12171.
* bootstrap.py was printing an improperly-cased path to put PGO/IBC
training data, which breaks on case-sensitive file systems, i.e., on
Linux.
* On Linux, `skiprestoreoptdata` was improperly skipping PGO even if
optdata was already present due to some of the conditional checks in
build.sh. This cleans that up to better match the intent of the
switch, and to fix Linux behavior to better match Windows.
Diffstat (limited to 'tests')
| -rwxr-xr-x | tests/scripts/optdata/bootstrap.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/scripts/optdata/bootstrap.py b/tests/scripts/optdata/bootstrap.py index 080fbf2a16..8fdf6d975b 100755 --- a/tests/scripts/optdata/bootstrap.py +++ b/tests/scripts/optdata/bootstrap.py @@ -51,7 +51,7 @@ def get_optdata_dir(tool, arch): """Returns an absolute path to the directory that should contain optdata given a tool,arch""" package_name = 'optimization.%s-%s.%s.CoreCLR' % (get_buildos(), arch, tool) package_version = get_optdata_version(tool) - return path.join(REPO_ROOT, 'packages', package_name, package_version, 'data') + return path.join(REPO_ROOT, 'packages', package_name.lower(), package_version.lower(), 'data') def check_for_unstaged_changes(file_path): """Returns whether a file in git has untracked changes.""" |