]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Rollup merge of #87191 - adamgemmell:dev/llvm-lib-package, r=Mark-Simulacrum
authorYuki Okushi <jtitor@2k36.org>
Fri, 23 Jul 2021 19:31:06 +0000 (04:31 +0900)
committerGitHub <noreply@github.com>
Fri, 23 Jul 2021 19:31:06 +0000 (04:31 +0900)
commitaca83f1ab84f43bb3f7345baf84719eb2da4644b
treeee22e2952c7ddf0925ab053eb6e2208d36486bc5
parentba869da8891627c78ef0f328d737e971c36da89e
parent24254d693b0f4d55c57c3e9a7df4cb8795d2d9d2
Rollup merge of #87191 - adamgemmell:dev/llvm-lib-package, r=Mark-Simulacrum

Package LLVM libs for the target rather than the build host

Fixes https://github.com/rust-lang/rust/issues/85250

`dist.rs` uses, in the `rust-dev` stage, `llvm-config --libfiles` to get a list of the LLVM library files built but of course only for the build host. If the target differs we want to package lib files from the target's build tree instead. This is done by splitting/rejoining the paths on their build directories.

At the moment `tree` on the LLVM build directories seems to give almost identical output, but of course this might not be the case in the future. If a file is missing in the target's build tree then this stage will error in the `builder.install()` call. If the target build tree has an extra file then it silently won't be copied and we'll get a linker error when building using this artifact (via `download-ci-llvm = "if-available"`), though we would have received a linker error anyway without this change.

There was also a typo in the example config around this option.