Skip to content

Commit

Permalink
Rollup merge of rust-lang#53154 - michaelwoerister:reenable-some-debu…
Browse files Browse the repository at this point in the history
…ginfo-tests, r=kennytm

Re-enable a bunch of debuginfo tests.

Re-enable some more debuginfo tests that actually seem to work.
  • Loading branch information
cramertj authored Aug 8, 2018
2 parents 42edec6 + 4eb52ff commit 197861a
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 11 deletions.
1 change: 0 additions & 1 deletion src/test/debuginfo/by-value-non-immediate-argument.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
// except according to those terms.

// ignore-tidy-linelength
// ignore-test // Test temporarily ignored due to debuginfo tests being disabled, see PR 47155
// min-lldb-version: 310

// compile-flags:-g
Expand Down
1 change: 0 additions & 1 deletion src/test/debuginfo/function-arg-initialization.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
// except according to those terms.

// ignore-tidy-linelength
// ignore-test // Test temporarily ignored due to debuginfo tests being disabled, see PR 47155
// min-lldb-version: 310

// This test case checks if function arguments already have the correct value
Expand Down
1 change: 0 additions & 1 deletion src/test/debuginfo/function-prologue-stepping-regular.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@

// min-lldb-version: 310
// ignore-gdb
// ignore-test // Test temporarily ignored due to debuginfo tests being disabled, see PR 47155
// compile-flags:-g

// lldb-command:breakpoint set --name immediate_args
Expand Down
16 changes: 8 additions & 8 deletions src/test/debuginfo/pretty-std.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@

// ignore-windows failing on win32 bot
// ignore-freebsd: gdb package too new
// ignore-test // Test temporarily ignored due to debuginfo tests being disabled, see PR 47155
// ignore-android: FIXME(#10381)
// compile-flags:-g
// min-gdb-version 7.7
Expand Down Expand Up @@ -40,14 +39,15 @@
// gdbr-check:$6 = core::option::Option::None

// gdb-command: print os_string
// gdb-check:$7 = "IAMA OS string 😃"
// gdb-check:$7 = "IAMA OS string"

// gdb-command: print some_string
// gdb-check:$8 = Some = {"IAMA optional string!"}
// Disabled due to https://github.com/rust-lang/rust/issues/53153
// g d b-command: print some_string
// g d b-check:$8 = Some = {"IAMA optional string!"}

// gdb-command: set print length 5
// gdb-command: print some_string
// gdb-check:$8 = Some = {"IAMA "...}
// g d b-command: set print length 5
// g d b-command: print some_string
// g d b-check:$8 = Some = {"IAMA "...}


// === LLDB TESTS ==================================================================================
Expand Down Expand Up @@ -92,7 +92,7 @@ fn main() {
let string = "IAMA string!".to_string();

// OsString
let os_string = OsString::from("IAMA OS string \u{1F603}");
let os_string = OsString::from("IAMA OS string");

// Option
let some = Some(8i16);
Expand Down

0 comments on commit 197861a

Please sign in to comment.