mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Auto merge of #113323 - Kobzol:pgo-script-llvm-ci, r=jyn514

Browse Source 
Use `llvm-config` instead of `download-ci-llvm` in PGO script

This should avoid CI breakage when the LLVM stamp is updated, and also it will avoid an unnecessary LLVM download from CI.

r? `@jyn514`
This commit is contained in:
bors 2023-07-06 13:01:13 +00:00
commit c4c84df3b3
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/ci/stage-build.py
View File

@ -840,6 +840,11 @@ def run_tests(pipeline: Pipeline):
cargo_path = cargo_dir / "bin" / f"cargo{pipeline.executable_extension()}" cargo_path = cargo_dir / "bin" / f"cargo{pipeline.executable_extension()}"
assert cargo_path.is_file() assert cargo_path.is_file()
# Specify path to a LLVM config so that LLVM is not rebuilt.
# It doesn't really matter which LLVM config we choose, because no sysroot will be compiled.
llvm_config = pipeline.build_artifacts() / "llvm" / "bin" / f"llvm-config{pipeline.executable_extension()}"
assert llvm_config.is_file()
config_content = f"""profile = "user" config_content = f"""profile = "user"
changelog-seen = 2 changelog-seen = 2
@ -847,8 +852,8 @@ changelog-seen = 2
rustc = "{rustc_path.as_posix()}" rustc = "{rustc_path.as_posix()}"
cargo = "{cargo_path.as_posix()}" cargo = "{cargo_path.as_posix()}"
[llvm] [target.{PGO_HOST}]
download-ci-llvm = true llvm-config = "{llvm_config.as_posix()}"
""" """
logging.info(f"Using following `config.toml` for running tests:\n{config_content}") logging.info(f"Using following `config.toml` for running tests:\n{config_content}")