Skip to content

Commit 64698bf

Browse files
committed
Test x86_64-gnu in PR CI, not x86_64-gnu-llvm-14
This is a pre-requisite for rust-lang#112143, which wants to start using download-rustc in PRs. download-rustc doesn't allow providing an external LLVM.
1 parent e94bda3 commit 64698bf

File tree

4 files changed

+26
-26
lines changed

4 files changed

+26
-26
lines changed

.github/workflows/ci.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
- name: mingw-check-tidy
5353
os: ubuntu-20.04-16core-64gb
5454
env: {}
55-
- name: x86_64-gnu-llvm-14
55+
- name: x86_64-gnu
5656
os: ubuntu-20.04-16core-64gb
5757
env: {}
5858
- name: x86_64-gnu-tools

src/ci/docker/host-x86_64/x86_64-gnu-llvm-14/Dockerfile

+1-23
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ ARG DEBIAN_FRONTEND=noninteractive
44

55
RUN apt-get update && apt-get install -y --no-install-recommends \
66
g++ \
7-
gcc-multilib \
87
make \
98
ninja-build \
109
file \
@@ -26,11 +25,6 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
2625
mingw-w64 \
2726
&& rm -rf /var/lib/apt/lists/*
2827

29-
# Install powershell (universal package) so we can test x.ps1 on Linux
30-
RUN curl -sL "https://github.com/PowerShell/PowerShell/releases/download/v7.3.1/powershell_7.3.1-1.deb_amd64.deb" > powershell.deb && \
31-
dpkg -i powershell.deb && \
32-
rm -f powershell.deb
33-
3428
COPY scripts/sccache.sh /scripts/
3529
RUN sh /scripts/sccache.sh
3630

@@ -49,20 +43,4 @@ ENV RUST_CONFIGURE_ARGS \
4943
--enable-llvm-link-shared \
5044
--set rust.thin-lto-import-instr-limit=10
5145

52-
# NOTE: intentionally uses all of `x.py`, `x`, and `x.ps1` to make sure they all work on Linux.
53-
ENV SCRIPT ../x.py --stage 2 test --exclude src/tools/tidy && \
54-
# Run the `mir-opt` tests again but this time for a 32-bit target.
55-
# This enforces that tests using `// EMIT_MIR_FOR_EACH_BIT_WIDTH` have
56-
# both 32-bit and 64-bit outputs updated by the PR author, before
57-
# the PR is approved and tested for merging.
58-
# It will also detect tests lacking `// EMIT_MIR_FOR_EACH_BIT_WIDTH`,
59-
# despite having different output on 32-bit vs 64-bit targets.
60-
../x --stage 2 test tests/mir-opt \
61-
--host='' --target=i686-unknown-linux-gnu && \
62-
# Run the UI test suite again, but in `--pass=check` mode
63-
#
64-
# This is intended to make sure that both `--pass=check` continues to
65-
# work.
66-
#
67-
../x.ps1 --stage 2 test tests/ui --pass=check \
68-
--host='' --target=i686-unknown-linux-gnu
46+
ENV SCRIPT ../x.py --stage 2 test

src/ci/docker/host-x86_64/x86_64-gnu/Dockerfile

+23-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ FROM ubuntu:20.04
33
ARG DEBIAN_FRONTEND=noninteractive
44
RUN apt-get update && apt-get install -y --no-install-recommends \
55
g++ \
6+
gcc-multilib \
67
make \
78
ninja-build \
89
file \
@@ -19,6 +20,11 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
1920
mingw-w64 \
2021
&& rm -rf /var/lib/apt/lists/*
2122

23+
# Install powershell (universal package) so we can test x.ps1 on Linux
24+
RUN curl -sL "https://github.com/PowerShell/PowerShell/releases/download/v7.3.1/powershell_7.3.1-1.deb_amd64.deb" > powershell.deb && \
25+
dpkg -i powershell.deb && \
26+
rm -f powershell.deb
27+
2228
COPY scripts/sccache.sh /scripts/
2329
RUN sh /scripts/sccache.sh
2430

@@ -27,4 +33,20 @@ ENV RUST_CONFIGURE_ARGS \
2733
--enable-sanitizers \
2834
--enable-profiler \
2935
--enable-compiler-docs
30-
ENV SCRIPT python3 ../x.py --stage 2 test
36+
37+
# NOTE: intentionally uses all of `x.py`, `x`, and `x.ps1` to make sure they all work on Linux.
38+
ENV SCRIPT ../x.py --stage 2 test --exclude src/tools/tidy && \
39+
# Run the `mir-opt` tests again but this time for a 32-bit target.
40+
# This enforces that tests using `// EMIT_MIR_FOR_EACH_BIT_WIDTH` have
41+
# both 32-bit and 64-bit outputs updated by the PR author, before
42+
# the PR is approved and tested for merging.
43+
# It will also detect tests lacking `// EMIT_MIR_FOR_EACH_BIT_WIDTH`,
44+
# despite having different output on 32-bit vs 64-bit targets.
45+
../x --stage 2 test tests/mir-opt \
46+
--host='' --target=i686-unknown-linux-gnu && \
47+
# Run the UI test suite again, but in `--pass=check` mode
48+
#
49+
# This is intended to make sure that both `--pass=check` continues to
50+
# work.
51+
../x.ps1 --stage 2 test tests/ui --pass=check \
52+
--host='' --target=i686-unknown-linux-gnu

src/ci/github-actions/ci.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ jobs:
308308
- name: mingw-check-tidy
309309
<<: *job-linux-16c
310310

311-
- name: x86_64-gnu-llvm-14
311+
- name: x86_64-gnu
312312
<<: *job-linux-16c
313313

314314
- name: x86_64-gnu-tools

0 commit comments

Comments
 (0)