Skip to content

Commit

Permalink
get rid of the --miri_host_target hack
Browse files Browse the repository at this point in the history
  • Loading branch information
RalfJung committed Sep 16, 2017
1 parent 605c82b commit 5a5aa49
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 16 deletions.
16 changes: 3 additions & 13 deletions miri/bin/miri.rs
Expand Up @@ -22,8 +22,6 @@ use std::path::PathBuf;

struct MiriCompilerCalls {
default: RustcDefaultCalls,
/// whether we are building for the host
host_target: bool,
}

impl<'a> CompilerCalls<'a> for MiriCompilerCalls {
Expand Down Expand Up @@ -80,8 +78,8 @@ impl<'a> CompilerCalls<'a> for MiriCompilerCalls {
let mut control = self.default.build_controller(sess, matches);
control.after_hir_lowering.callback = Box::new(after_hir_lowering);
control.after_analysis.callback = Box::new(after_analysis);
if !self.host_target {
// only fully compile targets on the host
if sess.target.target != sess.host {
// only fully compile targets on the host. linking will fail for cross-compilation.
control.after_analysis.stop = Compilation::Stop;
}
control
Expand Down Expand Up @@ -258,18 +256,10 @@ fn main() {
args.push(find_sysroot());
}

// for auxilary builds in unit tests
// Make sure we always have all the MIR (e.g. for auxilary builds in unit tests).
args.push("-Zalways-encode-mir".to_owned());
let mut host_target = false;
args.retain(|arg| if arg == "--miri_host_target" {
host_target = true;
false // remove the flag, rustc doesn't know it
} else {
true
});

rustc_driver::run_compiler(&args, &mut MiriCompilerCalls {
default: RustcDefaultCalls,
host_target,
}, None, None);
}
3 changes: 0 additions & 3 deletions tests/compiletest.rs
Expand Up @@ -122,9 +122,6 @@ fn miri_pass(path: &str, target: &str, host: &str, fullmir: bool, opt: bool) {
// For now, only validate without optimizations. Inlining breaks validation.
flags.push("-Zmir-emit-validate=1".to_owned());
}
if target == host {
flags.push("--miri_host_target".to_owned());
}
config.target_rustcflags = Some(flags.join(" "));
compiletest::run_tests(&config);
}
Expand Down

0 comments on commit 5a5aa49

Please sign in to comment.