ci: Begin running ui tests with rust.debuginfo-level-tests=1

To reduce risk of regressing on generating debuginfo e.g. in the form of
ICE:s. This will also ensure that future ui tests support different
debuginfo levels.

When I looked at run time for different CI jobs, **x86_64-gnu-debug**
was far from the bottle neck, so it should be fine to make it perform
more work.
This commit is contained in:
Martin Nordholts 2025-07-26 13:23:20 +02:00
parent f5703d5dd3
commit 9637774e32
9 changed files with 24 additions and 3 deletions

View file

@ -38,11 +38,15 @@ ENV RUST_CONFIGURE_ARGS \
--build=x86_64-unknown-linux-gnu \
--enable-debug \
--enable-lld \
--set rust.debuginfo-level-tests=1 \
--set llvm.use-linker=lld \
--set target.x86_64-unknown-linux-gnu.linker=clang \
--set target.x86_64-unknown-linux-gnu.cc=clang \
--set target.x86_64-unknown-linux-gnu.cxx=clang++
# This job checks:
# - That ui tests can be built with `-Cdebuginfo=1`
# This job appears to be checking two separate things:
# - That we can build the compiler with `--enable-debug`
# (without necessarily testing the result).
@ -51,4 +55,5 @@ ENV RUST_CONFIGURE_ARGS \
ENV SCRIPT \
python3 ../x.py --stage 2 build && \
python3 ../x.py --stage 2 test tests/ui && \
python3 ../x.py --stage 2 test tests/run-make

View file

@ -1,4 +1,8 @@
//@ check-pass
//@ ignore-windows
//@ compile-flags: -Cremark=foo --error-format=human --color=always
// FIXME(#61117): Respect debuginfo-level-tests, do not force debuginfo-level=0
//@ compile-flags: -Cdebuginfo=0
fn main() {}

View file

@ -2,6 +2,9 @@
//
//@ run-pass
// FIXME(#61117): Respect debuginfo-level-tests, do not force debuginfo-level=0
//@ compile-flags: -Cdebuginfo=0
#![allow(unused_variables)]
macro_rules! m(

View file

@ -2,6 +2,9 @@
//@ normalize-stderr: "std::option::Option<\[u32; \d+\]>" -> "TYPE"
//@ normalize-stderr: "\[u32; \d+\]" -> "TYPE"
// FIXME(#61117): Respect debuginfo-level-tests, do not force debuginfo-level=0
//@ compile-flags: -Cdebuginfo=0
#[cfg(target_pointer_width = "32")]
type BIG = Option<[u32; (1<<29)-1]>;

View file

@ -1,5 +1,5 @@
error: values of the type `Option<TYPE>` are too big for the target architecture
--> $DIR/huge-enum.rs:12:9
--> $DIR/huge-enum.rs:15:9
|
LL | let big: BIG = None;
| ^^^

View file

@ -6,6 +6,9 @@
//@ check-run-results
//@ exec-env:RUST_BACKTRACE=1
// FIXME(#61117): Respect debuginfo-level-tests, do not force debuginfo-level=0
//@ compile-flags: -Cdebuginfo=0
// This is needed to avoid test output differences across std being built with v0 symbols vs legacy
// symbols.
//@ normalize-stderr: "begin_panic::<&str>" -> "begin_panic"

View file

@ -1,5 +1,5 @@
thread 'main' ($TID) panicked at $DIR/issue-47429-short-backtraces.rs:24:5:
thread 'main' ($TID) panicked at $DIR/issue-47429-short-backtraces.rs:27:5:
explicit panic
stack backtrace:
0: std::panicking::begin_panic

View file

@ -6,6 +6,9 @@
//@ check-run-results
//@ exec-env:RUST_BACKTRACE=0
// FIXME(#61117): Respect debuginfo-level-tests, do not force debuginfo-level=0
//@ compile-flags: -Cdebuginfo=0
// This is needed to avoid test output differences across std being built with v0 symbols vs legacy
// symbols.
//@ normalize-stderr: "begin_panic::<&str>" -> "begin_panic"

View file

@ -1,5 +1,5 @@
thread 'main' ($TID) panicked at $DIR/runtime-switch.rs:28:5:
thread 'main' ($TID) panicked at $DIR/runtime-switch.rs:31:5:
explicit panic
stack backtrace:
0: std::panicking::begin_panic