Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/ci/docker/host-x86_64/x86_64-gnu-debug/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -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).
Expand All @@ -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
4 changes: 4 additions & 0 deletions tests/ui/diagnostic-flags/colored-session-opt-error.rs
Original file line number Diff line number Diff line change
@@ -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() {}
3 changes: 3 additions & 0 deletions tests/ui/issues/issue-29466.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down
3 changes: 3 additions & 0 deletions tests/ui/limits/huge-enum.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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]>;

Expand Down
2 changes: 1 addition & 1 deletion tests/ui/limits/huge-enum.stderr
Original file line number Diff line number Diff line change
@@ -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;
| ^^^
Expand Down
3 changes: 3 additions & 0 deletions tests/ui/panics/issue-47429-short-backtraces.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down
2 changes: 1 addition & 1 deletion tests/ui/panics/issue-47429-short-backtraces.run.stderr
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@

thread 'main' panicked at $DIR/issue-47429-short-backtraces.rs:24:5:
thread 'main' panicked at $DIR/issue-47429-short-backtraces.rs:27:5:
explicit panic
stack backtrace:
0: std::panicking::begin_panic
Expand Down
3 changes: 3 additions & 0 deletions tests/ui/panics/runtime-switch.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down
2 changes: 1 addition & 1 deletion tests/ui/panics/runtime-switch.run.stderr
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@

thread 'main' panicked at $DIR/runtime-switch.rs:28:5:
thread 'main' panicked at $DIR/runtime-switch.rs:31:5:
explicit panic
stack backtrace:
0: std::panicking::begin_panic
Expand Down
Loading