Skip to content

Commit

Permalink
Rollup merge of #102032 - andrewpollack:bad-signal-compiler-tests, r=…
Browse files Browse the repository at this point in the history
…tmandry

Adding ignore fuchsia tests for signal interpretation cases

Tests where Signal interpreting is required. Since Fuchsia currently does not return signals of type `libc::SIGSEGV` etc., instead, use generalized `!status.success()` case.

cc. `@djkoloski`

r? `@tmandry`
  • Loading branch information
JohnTitor authored Sep 22, 2022
2 parents 614e18b + 6c29716 commit 5542e50
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/test/ui/abi/segfault-no-out-of-stack.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
#![allow(unused_imports)]
// ignore-emscripten can't run commands
// ignore-sgx no processes
// ignore-fuchsia must translate zircon signal to SIGSEGV/SIGBUS, FIXME (#58590)
#![feature(rustc_private)]

extern crate libc;
Expand Down
1 change: 1 addition & 0 deletions src/test/ui/abi/stack-probes-lto.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
// ignore-sgx no processes
// ignore-musl FIXME #31506
// ignore-pretty
// ignore-fuchsia no exception handler registered for segfault
// compile-flags: -C lto
// no-prefer-dynamic

Expand Down
1 change: 1 addition & 0 deletions src/test/ui/abi/stack-probes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
// ignore-wasm
// ignore-emscripten no processes
// ignore-sgx no processes
// ignore-fuchsia no exception handler registered for segfault

use std::env;
use std::mem::MaybeUninit;
Expand Down
1 change: 1 addition & 0 deletions src/test/ui/process/signal-exit-status.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
// ignore-emscripten no processes
// ignore-sgx no processes
// ignore-windows
// ignore-fuchsia code returned as ZX_TASK_RETCODE_EXCEPTION_KILL, FIXME (#58590)

use std::env;
use std::process::Command;
Expand Down
1 change: 1 addition & 0 deletions src/test/ui/runtime/out-of-stack.rs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
// ignore-android: FIXME (#20004)
// ignore-emscripten no processes
// ignore-sgx no processes
// ignore-fuchsia must translate zircon signal to SIGABRT, FIXME (#58590)

#![feature(core_intrinsics)]
#![feature(rustc_private)]
Expand Down

0 comments on commit 5542e50

Please sign in to comment.