diff options
| author | Oli Scherer <git-spam-no-reply9815368754983@oli-obk.de> | 2022-09-21 09:49:30 +0000 |
|---|---|---|
| committer | Oli Scherer <git-spam-no-reply9815368754983@oli-obk.de> | 2022-09-21 12:48:47 +0000 |
| commit | 4f357956b868cd5d6c1a9ff0608dfcd7d5bbe2a3 (patch) | |
| tree | 0a34204ec6bd3f47c29708a6a4295c29704b9a49 | |
| parent | 369efd0d8e477166a6195334bd2ad6a18d41b267 (diff) | |
| download | rust-4f357956b868cd5d6c1a9ff0608dfcd7d5bbe2a3.tar.gz rust-4f357956b868cd5d6c1a9ff0608dfcd7d5bbe2a3.zip | |
Only support libffi on unix for now
| -rw-r--r-- | Cargo.toml | 6 | ||||
| -rw-r--r-- | README.md | 2 | ||||
| -rw-r--r-- | src/machine.rs | 4 | ||||
| -rw-r--r-- | src/shims/foreign_items.rs | 2 | ||||
| -rw-r--r-- | src/shims/mod.rs | 1 |
5 files changed, 9 insertions, 6 deletions
diff --git a/Cargo.toml b/Cargo.toml index 0a3dfc2a84e..0c547d585d1 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -20,8 +20,6 @@ doctest = false # and no doc tests [dependencies] getrandom = { version = "0.2", features = ["std"] } env_logger = "0.9" -libffi = "3.0.0" -libloading = "0.7" log = "0.4" shell-escape = "0.1.4" rand = "0.8" @@ -33,10 +31,10 @@ smallvec = "1.7" rustc-workspace-hack = "1.0.0" measureme = "10.0.0" -# Enable some feature flags that dev-dependencies need but dependencies -# do not. This makes `./miri install` after `./miri build` faster. [target."cfg(unix)".dependencies] libc = "0.2" +libffi = "3.0.0" +libloading = "0.7" [dev-dependencies] colored = "2" diff --git a/README.md b/README.md index 72081f45499..1f4c52d5b85 100644 --- a/README.md +++ b/README.md @@ -360,7 +360,7 @@ to Miri failing to detect cases of undefined behavior in a program. file descriptors will be mixed up. This is **work in progress**; currently, only integer arguments and return values are supported (and no, pointer/integer casts to work around this limitation will not work; - they will fail horribly). + they will fail horribly). It also only works on unix hosts for now. Follow [the discussion on supporting other types](https://github.com/rust-lang/miri/issues/2365). * `-Zmiri-measureme=<name>` enables `measureme` profiling for the interpreted program. This can be used to find which parts of your program are executing slowly under Miri. diff --git a/src/machine.rs b/src/machine.rs index 8db546463c2..f11771911fa 100644 --- a/src/machine.rs +++ b/src/machine.rs @@ -400,6 +400,7 @@ pub struct MiriMachine<'mir, 'tcx> { pub(crate) basic_block_count: u64, /// Handle of the optional shared object file for external functions. + #[cfg(unix)] pub external_so_lib: Option<(libloading::Library, std::path::PathBuf)>, /// Run a garbage collector for SbTags every N basic blocks. @@ -410,7 +411,6 @@ pub struct MiriMachine<'mir, 'tcx> { impl<'mir, 'tcx> MiriMachine<'mir, 'tcx> { pub(crate) fn new(config: &MiriConfig, layout_cx: LayoutCx<'tcx, TyCtxt<'tcx>>) -> Self { - let target_triple = &layout_cx.tcx.sess.opts.target_triple.to_string(); let local_crates = helpers::get_local_crates(layout_cx.tcx); let layouts = PrimitiveLayouts::new(layout_cx).expect("Couldn't get layouts of primitive types"); @@ -462,7 +462,9 @@ impl<'mir, 'tcx> MiriMachine<'mir, 'tcx> { report_progress: config.report_progress, basic_block_count: 0, clock: Clock::new(config.isolated_op == IsolatedOp::Allow), + #[cfg(unix)] external_so_lib: config.external_so_file.as_ref().map(|lib_file_path| { + let target_triple = &layout_cx.tcx.sess.opts.target_triple.to_string(); // Check if host target == the session target. if env!("TARGET") != target_triple { panic!( diff --git a/src/shims/foreign_items.rs b/src/shims/foreign_items.rs index 6d052083d37..bb62a2a7ec1 100644 --- a/src/shims/foreign_items.rs +++ b/src/shims/foreign_items.rs @@ -23,6 +23,7 @@ use rustc_target::{ use super::backtrace::EvalContextExt as _; use crate::helpers::{convert::Truncate, target_os_is_unix}; +#[cfg(unix)] use crate::shims::ffi_support::EvalContextExt as _; use crate::*; @@ -371,6 +372,7 @@ pub trait EvalContextExt<'mir, 'tcx: 'mir>: crate::MiriInterpCxExt<'mir, 'tcx> { let this = self.eval_context_mut(); // First deal with any external C functions in linked .so file. + #[cfg(unix)] if this.machine.external_so_lib.as_ref().is_some() { // An Ok(false) here means that the function being called was not exported // by the specified `.so` file; we should continue and check if it corresponds to diff --git a/src/shims/mod.rs b/src/shims/mod.rs index 93083b486bf..8cb648e5173 100644 --- a/src/shims/mod.rs +++ b/src/shims/mod.rs @@ -1,6 +1,7 @@ #![warn(clippy::integer_arithmetic)] mod backtrace; +#[cfg(unix)] pub mod ffi_support; pub mod foreign_items; pub mod intrinsics; |
