Auto merge of #49542 - GuillaumeGomez:intra-link-resolution-error, r=GuillaumeGomez
Add warning if a resolution failed r? @QuietMisdreavus
This commit is contained in:
commit
8728c7a726
|
@ -326,7 +326,7 @@ impl<'a> Builder<'a> {
|
|||
test::TheBook, test::UnstableBook,
|
||||
test::Rustfmt, test::Miri, test::Clippy, test::RustdocJS, test::RustdocTheme,
|
||||
// Run run-make last, since these won't pass without make on Windows
|
||||
test::RunMake),
|
||||
test::RunMake, test::RustdocUi),
|
||||
Kind::Bench => describe!(test::Crate, test::CrateLibrustc),
|
||||
Kind::Doc => describe!(doc::UnstableBook, doc::UnstableBookGen, doc::TheBook,
|
||||
doc::Standalone, doc::Std, doc::Test, doc::WhitelistedRustc, doc::Rustc,
|
||||
|
|
|
@ -514,6 +514,41 @@ impl Step for RustdocJS {
|
|||
}
|
||||
}
|
||||
|
||||
#[derive(Debug, Copy, Clone, Hash, PartialEq, Eq)]
|
||||
pub struct RustdocUi {
|
||||
pub host: Interned<String>,
|
||||
pub target: Interned<String>,
|
||||
pub compiler: Compiler,
|
||||
}
|
||||
|
||||
impl Step for RustdocUi {
|
||||
type Output = ();
|
||||
const DEFAULT: bool = true;
|
||||
const ONLY_HOSTS: bool = true;
|
||||
|
||||
fn should_run(run: ShouldRun) -> ShouldRun {
|
||||
run.path("src/test/rustdoc-ui")
|
||||
}
|
||||
|
||||
fn make_run(run: RunConfig) {
|
||||
let compiler = run.builder.compiler(run.builder.top_stage, run.host);
|
||||
run.builder.ensure(RustdocUi {
|
||||
host: run.host,
|
||||
target: run.target,
|
||||
compiler,
|
||||
});
|
||||
}
|
||||
|
||||
fn run(self, builder: &Builder) {
|
||||
builder.ensure(Compiletest {
|
||||
compiler: self.compiler,
|
||||
target: self.target,
|
||||
mode: "ui",
|
||||
suite: "rustdoc-ui",
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
#[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)]
|
||||
pub struct Tidy;
|
||||
|
||||
|
@ -851,8 +886,12 @@ impl Step for Compiletest {
|
|||
cmd.arg("--run-lib-path").arg(builder.sysroot_libdir(compiler, target));
|
||||
cmd.arg("--rustc-path").arg(builder.rustc(compiler));
|
||||
|
||||
let is_rustdoc_ui = suite.ends_with("rustdoc-ui");
|
||||
|
||||
// Avoid depending on rustdoc when we don't need it.
|
||||
if mode == "rustdoc" || (mode == "run-make" && suite.ends_with("fulldeps")) {
|
||||
if mode == "rustdoc" ||
|
||||
(mode == "run-make" && suite.ends_with("fulldeps")) ||
|
||||
(mode == "ui" && is_rustdoc_ui) {
|
||||
cmd.arg("--rustdoc-path").arg(builder.rustdoc(compiler.host));
|
||||
}
|
||||
|
||||
|
@ -868,12 +907,18 @@ impl Step for Compiletest {
|
|||
cmd.arg("--nodejs").arg(nodejs);
|
||||
}
|
||||
|
||||
let mut flags = vec!["-Crpath".to_string()];
|
||||
if build.config.rust_optimize_tests {
|
||||
flags.push("-O".to_string());
|
||||
}
|
||||
if build.config.rust_debuginfo_tests {
|
||||
flags.push("-g".to_string());
|
||||
let mut flags = if is_rustdoc_ui {
|
||||
Vec::new()
|
||||
} else {
|
||||
vec!["-Crpath".to_string()]
|
||||
};
|
||||
if !is_rustdoc_ui {
|
||||
if build.config.rust_optimize_tests {
|
||||
flags.push("-O".to_string());
|
||||
}
|
||||
if build.config.rust_debuginfo_tests {
|
||||
flags.push("-g".to_string());
|
||||
}
|
||||
}
|
||||
flags.push("-Zunstable-options".to_string());
|
||||
flags.push(build.config.cmd.rustc_args().join(" "));
|
||||
|
|
|
@ -998,7 +998,7 @@ pub trait Iterator {
|
|||
/// an extra layer of indirection. `flat_map()` will remove this extra layer
|
||||
/// on its own.
|
||||
///
|
||||
/// You can think of [`flat_map(f)`][flat_map] as the semantic equivalent
|
||||
/// You can think of `flat_map(f)` as the semantic equivalent
|
||||
/// of [`map`]ping, and then [`flatten`]ing as in `map(f).flatten()`.
|
||||
///
|
||||
/// Another way of thinking about `flat_map()`: [`map`]'s closure returns
|
||||
|
|
|
@ -258,7 +258,7 @@ pub struct CharSearcher<'a> {
|
|||
|
||||
/// `finger` is the current byte index of the forward search.
|
||||
/// Imagine that it exists before the byte at its index, i.e.
|
||||
/// haystack[finger] is the first byte of the slice we must inspect during
|
||||
/// `haystack[finger]` is the first byte of the slice we must inspect during
|
||||
/// forward searching
|
||||
finger: usize,
|
||||
/// `finger_back` is the current byte index of the reverse search.
|
||||
|
|
|
@ -1550,7 +1550,7 @@ impl<'a, 'gcx, 'tcx> TyS<'tcx> {
|
|||
}
|
||||
}
|
||||
|
||||
/// Returns the type of ty[i]
|
||||
/// Returns the type of `ty[i]`.
|
||||
pub fn builtin_index(&self) -> Option<Ty<'tcx>> {
|
||||
match self.sty {
|
||||
TyArray(ty, _) | TySlice(ty) => Some(ty),
|
||||
|
|
|
@ -1178,6 +1178,10 @@ enum PathKind {
|
|||
Type,
|
||||
}
|
||||
|
||||
fn resolution_failure(cx: &DocContext, path_str: &str) {
|
||||
cx.sess().warn(&format!("[{}] cannot be resolved, ignoring it...", path_str));
|
||||
}
|
||||
|
||||
impl Clean<Attributes> for [ast::Attribute] {
|
||||
fn clean(&self, cx: &DocContext) -> Attributes {
|
||||
let mut attrs = Attributes::from_ast(cx.sess().diagnostic(), self);
|
||||
|
@ -1228,6 +1232,7 @@ impl Clean<Attributes> for [ast::Attribute] {
|
|||
if let Ok(def) = resolve(cx, path_str, true) {
|
||||
def
|
||||
} else {
|
||||
resolution_failure(cx, path_str);
|
||||
// this could just be a normal link or a broken link
|
||||
// we could potentially check if something is
|
||||
// "intra-doc-link-like" and warn in that case
|
||||
|
@ -1238,6 +1243,7 @@ impl Clean<Attributes> for [ast::Attribute] {
|
|||
if let Ok(def) = resolve(cx, path_str, false) {
|
||||
def
|
||||
} else {
|
||||
resolution_failure(cx, path_str);
|
||||
// this could just be a normal link
|
||||
continue;
|
||||
}
|
||||
|
@ -1282,6 +1288,7 @@ impl Clean<Attributes> for [ast::Attribute] {
|
|||
} else if let Ok(value_def) = resolve(cx, path_str, true) {
|
||||
value_def
|
||||
} else {
|
||||
resolution_failure(cx, path_str);
|
||||
// this could just be a normal link
|
||||
continue;
|
||||
}
|
||||
|
@ -1290,6 +1297,7 @@ impl Clean<Attributes> for [ast::Attribute] {
|
|||
if let Some(def) = macro_resolve(cx, path_str) {
|
||||
(def, None)
|
||||
} else {
|
||||
resolution_failure(cx, path_str);
|
||||
continue
|
||||
}
|
||||
}
|
||||
|
|
|
@ -18,6 +18,7 @@ use rustc::middle::privacy::AccessLevels;
|
|||
use rustc::ty::{self, TyCtxt, AllArenas};
|
||||
use rustc::hir::map as hir_map;
|
||||
use rustc::lint;
|
||||
use rustc::session::config::ErrorOutputType;
|
||||
use rustc::util::nodemap::{FxHashMap, FxHashSet};
|
||||
use rustc_resolve as resolve;
|
||||
use rustc_metadata::creader::CrateLoader;
|
||||
|
@ -28,8 +29,9 @@ use syntax::ast::NodeId;
|
|||
use syntax::codemap;
|
||||
use syntax::edition::Edition;
|
||||
use syntax::feature_gate::UnstableFeatures;
|
||||
use syntax::json::JsonEmitter;
|
||||
use errors;
|
||||
use errors::emitter::ColorConfig;
|
||||
use errors::emitter::{Emitter, EmitterWriter};
|
||||
|
||||
use std::cell::{RefCell, Cell};
|
||||
use std::mem;
|
||||
|
@ -115,7 +117,6 @@ impl DocAccessLevels for AccessLevels<DefId> {
|
|||
}
|
||||
}
|
||||
|
||||
|
||||
pub fn run_core(search_paths: SearchPaths,
|
||||
cfgs: Vec<String>,
|
||||
externs: config::Externs,
|
||||
|
@ -126,7 +127,8 @@ pub fn run_core(search_paths: SearchPaths,
|
|||
crate_name: Option<String>,
|
||||
force_unstable_if_unmarked: bool,
|
||||
edition: Edition,
|
||||
cg: CodegenOptions) -> (clean::Crate, RenderInfo)
|
||||
cg: CodegenOptions,
|
||||
error_format: ErrorOutputType) -> (clean::Crate, RenderInfo)
|
||||
{
|
||||
// Parse, resolve, and typecheck the given crate.
|
||||
|
||||
|
@ -138,6 +140,7 @@ pub fn run_core(search_paths: SearchPaths,
|
|||
let warning_lint = lint::builtin::WARNINGS.name_lower();
|
||||
|
||||
let host_triple = TargetTriple::from_triple(config::host_triple());
|
||||
// plays with error output here!
|
||||
let sessopts = config::Options {
|
||||
maybe_sysroot,
|
||||
search_paths,
|
||||
|
@ -155,14 +158,42 @@ pub fn run_core(search_paths: SearchPaths,
|
|||
edition,
|
||||
..config::basic_debugging_options()
|
||||
},
|
||||
error_format,
|
||||
..config::basic_options().clone()
|
||||
};
|
||||
|
||||
let codemap = Lrc::new(codemap::CodeMap::new(sessopts.file_path_mapping()));
|
||||
let diagnostic_handler = errors::Handler::with_tty_emitter(ColorConfig::Auto,
|
||||
true,
|
||||
false,
|
||||
Some(codemap.clone()));
|
||||
let emitter: Box<dyn Emitter> = match error_format {
|
||||
ErrorOutputType::HumanReadable(color_config) => Box::new(
|
||||
EmitterWriter::stderr(
|
||||
color_config,
|
||||
Some(codemap.clone()),
|
||||
false,
|
||||
sessopts.debugging_opts.teach,
|
||||
).ui_testing(sessopts.debugging_opts.ui_testing)
|
||||
),
|
||||
ErrorOutputType::Json(pretty) => Box::new(
|
||||
JsonEmitter::stderr(
|
||||
None,
|
||||
codemap.clone(),
|
||||
pretty,
|
||||
sessopts.debugging_opts.approximate_suggestions,
|
||||
).ui_testing(sessopts.debugging_opts.ui_testing)
|
||||
),
|
||||
ErrorOutputType::Short(color_config) => Box::new(
|
||||
EmitterWriter::stderr(color_config, Some(codemap.clone()), true, false)
|
||||
),
|
||||
};
|
||||
|
||||
let diagnostic_handler = errors::Handler::with_emitter_and_flags(
|
||||
emitter,
|
||||
errors::HandlerFlags {
|
||||
can_emit_warnings: true,
|
||||
treat_err_as_bug: false,
|
||||
external_macro_backtrace: false,
|
||||
..Default::default()
|
||||
},
|
||||
);
|
||||
|
||||
let mut sess = session::build_session_(
|
||||
sessopts, cpath, diagnostic_handler, codemap,
|
||||
|
|
|
@ -107,7 +107,7 @@ pub struct SharedContext {
|
|||
/// This describes the layout of each page, and is not modified after
|
||||
/// creation of the context (contains info like the favicon and added html).
|
||||
pub layout: layout::Layout,
|
||||
/// This flag indicates whether [src] links should be generated or not. If
|
||||
/// This flag indicates whether `[src]` links should be generated or not. If
|
||||
/// the source files are present in the html rendering, then this will be
|
||||
/// `true`.
|
||||
pub include_sources: bool,
|
||||
|
|
|
@ -23,6 +23,7 @@
|
|||
#![feature(test)]
|
||||
#![feature(vec_remove_item)]
|
||||
#![feature(entry_and_modify)]
|
||||
#![feature(dyn_trait)]
|
||||
|
||||
extern crate arena;
|
||||
extern crate getopts;
|
||||
|
@ -48,6 +49,8 @@ extern crate tempdir;
|
|||
|
||||
extern crate serialize as rustc_serialize; // used by deriving
|
||||
|
||||
use errors::ColorConfig;
|
||||
|
||||
use std::collections::{BTreeMap, BTreeSet};
|
||||
use std::default::Default;
|
||||
use std::env;
|
||||
|
@ -279,6 +282,21 @@ pub fn opts() -> Vec<RustcOptGroup> {
|
|||
"edition to use when compiling rust code (default: 2015)",
|
||||
"EDITION")
|
||||
}),
|
||||
unstable("color", |o| {
|
||||
o.optopt("",
|
||||
"color",
|
||||
"Configure coloring of output:
|
||||
auto = colorize, if output goes to a tty (default);
|
||||
always = always colorize output;
|
||||
never = never colorize output",
|
||||
"auto|always|never")
|
||||
}),
|
||||
unstable("error-format", |o| {
|
||||
o.optopt("",
|
||||
"error-format",
|
||||
"How errors and other messages are produced",
|
||||
"human|json|short")
|
||||
}),
|
||||
]
|
||||
}
|
||||
|
||||
|
@ -363,9 +381,33 @@ pub fn main_args(args: &[String]) -> isize {
|
|||
}
|
||||
let input = &matches.free[0];
|
||||
|
||||
let color = match matches.opt_str("color").as_ref().map(|s| &s[..]) {
|
||||
Some("auto") => ColorConfig::Auto,
|
||||
Some("always") => ColorConfig::Always,
|
||||
Some("never") => ColorConfig::Never,
|
||||
None => ColorConfig::Auto,
|
||||
Some(arg) => {
|
||||
print_error(&format!("argument for --color must be `auto`, `always` or `never` \
|
||||
(instead was `{}`)", arg));
|
||||
return 1;
|
||||
}
|
||||
};
|
||||
let error_format = match matches.opt_str("error-format").as_ref().map(|s| &s[..]) {
|
||||
Some("human") => ErrorOutputType::HumanReadable(color),
|
||||
Some("json") => ErrorOutputType::Json(false),
|
||||
Some("pretty-json") => ErrorOutputType::Json(true),
|
||||
Some("short") => ErrorOutputType::Short(color),
|
||||
None => ErrorOutputType::HumanReadable(color),
|
||||
Some(arg) => {
|
||||
print_error(&format!("argument for --error-format must be `human`, `json` or \
|
||||
`short` (instead was `{}`)", arg));
|
||||
return 1;
|
||||
}
|
||||
};
|
||||
|
||||
let mut libs = SearchPaths::new();
|
||||
for s in &matches.opt_strs("L") {
|
||||
libs.add_path(s, ErrorOutputType::default());
|
||||
libs.add_path(s, error_format);
|
||||
}
|
||||
let externs = match parse_externs(&matches) {
|
||||
Ok(ex) => ex,
|
||||
|
@ -465,7 +507,9 @@ pub fn main_args(args: &[String]) -> isize {
|
|||
}
|
||||
|
||||
let output_format = matches.opt_str("w");
|
||||
let res = acquire_input(PathBuf::from(input), externs, edition, cg, &matches, move |out| {
|
||||
|
||||
let res = acquire_input(PathBuf::from(input), externs, edition, cg, &matches, error_format,
|
||||
move |out| {
|
||||
let Output { krate, passes, renderinfo } = out;
|
||||
info!("going to format");
|
||||
match output_format.as_ref().map(|s| &**s) {
|
||||
|
@ -509,13 +553,14 @@ fn acquire_input<R, F>(input: PathBuf,
|
|||
edition: Edition,
|
||||
cg: CodegenOptions,
|
||||
matches: &getopts::Matches,
|
||||
error_format: ErrorOutputType,
|
||||
f: F)
|
||||
-> Result<R, String>
|
||||
where R: 'static + Send, F: 'static + Send + FnOnce(Output) -> R {
|
||||
match matches.opt_str("r").as_ref().map(|s| &**s) {
|
||||
Some("rust") => Ok(rust_input(input, externs, edition, cg, matches, f)),
|
||||
Some("rust") => Ok(rust_input(input, externs, edition, cg, matches, error_format, f)),
|
||||
Some(s) => Err(format!("unknown input format: {}", s)),
|
||||
None => Ok(rust_input(input, externs, edition, cg, matches, f))
|
||||
None => Ok(rust_input(input, externs, edition, cg, matches, error_format, f))
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -546,6 +591,7 @@ fn rust_input<R, F>(cratefile: PathBuf,
|
|||
edition: Edition,
|
||||
cg: CodegenOptions,
|
||||
matches: &getopts::Matches,
|
||||
error_format: ErrorOutputType,
|
||||
f: F) -> R
|
||||
where R: 'static + Send,
|
||||
F: 'static + Send + FnOnce(Output) -> R
|
||||
|
@ -598,7 +644,7 @@ where R: 'static + Send,
|
|||
let (mut krate, renderinfo) =
|
||||
core::run_core(paths, cfgs, externs, Input::File(cratefile), triple, maybe_sysroot,
|
||||
display_warnings, crate_name.clone(),
|
||||
force_unstable_if_unmarked, edition, cg);
|
||||
force_unstable_if_unmarked, edition, cg, error_format);
|
||||
|
||||
info!("finished with rustc");
|
||||
|
||||
|
|
|
@ -61,7 +61,7 @@ use sys_common::{AsInner, IntoInner, FromInner};
|
|||
/// # Conversions
|
||||
///
|
||||
/// See the [module's toplevel documentation about conversions][conversions] for a discussion on
|
||||
/// the traits which `OsString` implements for conversions from/to native representations.
|
||||
/// the traits which `OsString` implements for [conversions] from/to native representations.
|
||||
///
|
||||
/// [`OsStr`]: struct.OsStr.html
|
||||
/// [`&OsStr`]: struct.OsStr.html
|
||||
|
@ -74,6 +74,7 @@ use sys_common::{AsInner, IntoInner, FromInner};
|
|||
/// [`new`]: #method.new
|
||||
/// [`push`]: #method.push
|
||||
/// [`as_os_str`]: #method.as_os_str
|
||||
/// [conversions]: index.html#conversions
|
||||
#[derive(Clone)]
|
||||
#[stable(feature = "rust1", since = "1.0.0")]
|
||||
pub struct OsString {
|
||||
|
@ -89,7 +90,7 @@ pub struct OsString {
|
|||
/// references; the latter are owned strings.
|
||||
///
|
||||
/// See the [module's toplevel documentation about conversions][conversions] for a discussion on
|
||||
/// the traits which `OsStr` implements for conversions from/to native representations.
|
||||
/// the traits which `OsStr` implements for [conversions] from/to native representations.
|
||||
///
|
||||
/// [`OsString`]: struct.OsString.html
|
||||
/// [`&str`]: ../primitive.str.html
|
||||
|
|
|
@ -44,10 +44,10 @@
|
|||
//!
|
||||
//! Once you are familiar with the contents of the standard library you may
|
||||
//! begin to find the verbosity of the prose distracting. At this stage in your
|
||||
//! development you may want to press the **[-]** button near the top of the
|
||||
//! development you may want to press the `[-]` button near the top of the
|
||||
//! page to collapse it into a more skimmable view.
|
||||
//!
|
||||
//! While you are looking at that **[-]** button also notice the **[src]**
|
||||
//! While you are looking at that `[-]` button also notice the `[src]`
|
||||
//! button. Rust's API documentation comes with the source code and you are
|
||||
//! encouraged to read it. The standard library source is generally high
|
||||
//! quality and a peek behind the curtains is often enlightening.
|
||||
|
|
|
@ -787,13 +787,13 @@ pub mod builtin {
|
|||
}
|
||||
}
|
||||
|
||||
/// A macro for defining #[cfg] if-else statements.
|
||||
/// A macro for defining `#[cfg]` if-else statements.
|
||||
///
|
||||
/// This is similar to the `if/elif` C preprocessor macro by allowing definition
|
||||
/// of a cascade of `#[cfg]` cases, emitting the implementation which matches
|
||||
/// first.
|
||||
///
|
||||
/// This allows you to conveniently provide a long list #[cfg]'d blocks of code
|
||||
/// This allows you to conveniently provide a long list `#[cfg]`'d blocks of code
|
||||
/// without having to rewrite each clause multiple times.
|
||||
macro_rules! cfg_if {
|
||||
($(
|
||||
|
|
|
@ -8,9 +8,9 @@
|
|||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
//! Machinery for hygienic macros, inspired by the MTWT[1] paper.
|
||||
//! Machinery for hygienic macros, inspired by the `MTWT[1]` paper.
|
||||
//!
|
||||
//! [1] Matthew Flatt, Ryan Culpepper, David Darais, and Robert Bruce Findler. 2012.
|
||||
//! `[1]` Matthew Flatt, Ryan Culpepper, David Darais, and Robert Bruce Findler. 2012.
|
||||
//! *Macros that work together: Compile-time bindings, partial expansion,
|
||||
//! and definition contexts*. J. Funct. Program. 22, 2 (March 2012), 181-216.
|
||||
//! DOI=10.1017/S0956796812000093 <http://dx.doi.org/10.1017/S0956796812000093>
|
||||
|
|
|
@ -1288,7 +1288,6 @@ fn get_concurrency() -> usize {
|
|||
|
||||
pub fn filter_tests(opts: &TestOpts, tests: Vec<TestDescAndFn>) -> Vec<TestDescAndFn> {
|
||||
let mut filtered = tests;
|
||||
|
||||
// Remove tests that don't match the test filter
|
||||
filtered = match opts.filter {
|
||||
None => filtered,
|
||||
|
|
|
@ -8,13 +8,13 @@
|
|||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
/// A macro for defining #[cfg] if-else statements.
|
||||
/// A macro for defining `#[cfg]` if-else statements.
|
||||
///
|
||||
/// This is similar to the `if/elif` C preprocessor macro by allowing definition
|
||||
/// of a cascade of `#[cfg]` cases, emitting the implementation which matches
|
||||
/// first.
|
||||
///
|
||||
/// This allows you to conveniently provide a long list #[cfg]'d blocks of code
|
||||
/// This allows you to conveniently provide a long list `#[cfg]`'d blocks of code
|
||||
/// without having to rewrite each clause multiple times.
|
||||
macro_rules! cfg_if {
|
||||
($(
|
||||
|
|
|
@ -8,7 +8,7 @@
|
|||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
//`#[cfg]` on struct field permits empty unusable struct
|
||||
// `#[cfg]` on struct field permits empty unusable struct
|
||||
|
||||
struct S {
|
||||
#[cfg(untrue)]
|
||||
|
|
|
@ -0,0 +1,17 @@
|
|||
// Copyright 2018 The Rust Project Developers. See the COPYRIGHT
|
||||
// file at the top-level directory of this distribution and at
|
||||
// http://rust-lang.org/COPYRIGHT.
|
||||
//
|
||||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
||||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
||||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
||||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
// compile-pass
|
||||
|
||||
//! Test with [Foo::baz], [Bar::foo], [Uniooon::X]
|
||||
|
||||
pub struct Foo {
|
||||
pub bar: usize,
|
||||
}
|
|
@ -0,0 +1,6 @@
|
|||
warning: [Foo::baz] cannot be resolved, ignoring it...
|
||||
|
||||
warning: [Bar::foo] cannot be resolved, ignoring it...
|
||||
|
||||
warning: [Uniooon::X] cannot be resolved, ignoring it...
|
||||
|
|
@ -283,6 +283,8 @@ pub fn parse_config(args: Vec<String>) -> Config {
|
|||
),
|
||||
};
|
||||
|
||||
let src_base = opt_path(matches, "src-base");
|
||||
let run_ignored = matches.opt_present("ignored");
|
||||
Config {
|
||||
compile_lib_path: make_absolute(opt_path(matches, "compile-lib-path")),
|
||||
run_lib_path: make_absolute(opt_path(matches, "run-lib-path")),
|
||||
|
@ -293,7 +295,7 @@ pub fn parse_config(args: Vec<String>) -> Config {
|
|||
valgrind_path: matches.opt_str("valgrind-path"),
|
||||
force_valgrind: matches.opt_present("force-valgrind"),
|
||||
llvm_filecheck: matches.opt_str("llvm-filecheck").map(|s| PathBuf::from(&s)),
|
||||
src_base: opt_path(matches, "src-base"),
|
||||
src_base,
|
||||
build_base: opt_path(matches, "build-base"),
|
||||
stage_id: matches.opt_str("stage-id").unwrap(),
|
||||
mode: matches
|
||||
|
@ -301,7 +303,7 @@ pub fn parse_config(args: Vec<String>) -> Config {
|
|||
.unwrap()
|
||||
.parse()
|
||||
.expect("invalid mode"),
|
||||
run_ignored: matches.opt_present("ignored"),
|
||||
run_ignored,
|
||||
filter: matches.free.first().cloned(),
|
||||
filter_exact: matches.opt_present("exact"),
|
||||
logfile: matches.opt_str("logfile").map(|s| PathBuf::from(&s)),
|
||||
|
|
|
@ -1288,7 +1288,9 @@ impl<'test> TestCx<'test> {
|
|||
// want to actually assert warnings about all this code. Instead
|
||||
// let's just ignore unused code warnings by defaults and tests
|
||||
// can turn it back on if needed.
|
||||
rustc.args(&["-A", "unused"]);
|
||||
if !self.config.src_base.ends_with("rustdoc-ui") {
|
||||
rustc.args(&["-A", "unused"]);
|
||||
}
|
||||
}
|
||||
_ => {}
|
||||
}
|
||||
|
@ -1582,7 +1584,12 @@ impl<'test> TestCx<'test> {
|
|||
}
|
||||
|
||||
fn make_compile_args(&self, input_file: &Path, output_file: TargetLocation) -> Command {
|
||||
let mut rustc = Command::new(&self.config.rustc_path);
|
||||
let is_rustdoc = self.config.src_base.ends_with("rustdoc-ui");
|
||||
let mut rustc = if !is_rustdoc {
|
||||
Command::new(&self.config.rustc_path)
|
||||
} else {
|
||||
Command::new(&self.config.rustdoc_path.clone().expect("no rustdoc built yet"))
|
||||
};
|
||||
rustc.arg(input_file).arg("-L").arg(&self.config.build_base);
|
||||
|
||||
// Optionally prevent default --target if specified in test compile-flags.
|
||||
|
@ -1605,17 +1612,19 @@ impl<'test> TestCx<'test> {
|
|||
rustc.args(&["--cfg", revision]);
|
||||
}
|
||||
|
||||
if let Some(ref incremental_dir) = self.props.incremental_dir {
|
||||
rustc.args(&[
|
||||
"-C",
|
||||
&format!("incremental={}", incremental_dir.display()),
|
||||
]);
|
||||
rustc.args(&["-Z", "incremental-verify-ich"]);
|
||||
rustc.args(&["-Z", "incremental-queries"]);
|
||||
}
|
||||
if !is_rustdoc {
|
||||
if let Some(ref incremental_dir) = self.props.incremental_dir {
|
||||
rustc.args(&[
|
||||
"-C",
|
||||
&format!("incremental={}", incremental_dir.display()),
|
||||
]);
|
||||
rustc.args(&["-Z", "incremental-verify-ich"]);
|
||||
rustc.args(&["-Z", "incremental-queries"]);
|
||||
}
|
||||
|
||||
if self.config.mode == CodegenUnits {
|
||||
rustc.args(&["-Z", "human_readable_cgu_names"]);
|
||||
if self.config.mode == CodegenUnits {
|
||||
rustc.args(&["-Z", "human_readable_cgu_names"]);
|
||||
}
|
||||
}
|
||||
|
||||
match self.config.mode {
|
||||
|
@ -1668,11 +1677,12 @@ impl<'test> TestCx<'test> {
|
|||
}
|
||||
}
|
||||
|
||||
|
||||
if self.config.target == "wasm32-unknown-unknown" {
|
||||
// rustc.arg("-g"); // get any backtrace at all on errors
|
||||
} else if !self.props.no_prefer_dynamic {
|
||||
rustc.args(&["-C", "prefer-dynamic"]);
|
||||
if !is_rustdoc {
|
||||
if self.config.target == "wasm32-unknown-unknown" {
|
||||
// rustc.arg("-g"); // get any backtrace at all on errors
|
||||
} else if !self.props.no_prefer_dynamic {
|
||||
rustc.args(&["-C", "prefer-dynamic"]);
|
||||
}
|
||||
}
|
||||
|
||||
match output_file {
|
||||
|
@ -1696,8 +1706,10 @@ impl<'test> TestCx<'test> {
|
|||
} else {
|
||||
rustc.args(self.split_maybe_args(&self.config.target_rustcflags));
|
||||
}
|
||||
if let Some(ref linker) = self.config.linker {
|
||||
rustc.arg(format!("-Clinker={}", linker));
|
||||
if !is_rustdoc {
|
||||
if let Some(ref linker) = self.config.linker {
|
||||
rustc.arg(format!("-Clinker={}", linker));
|
||||
}
|
||||
}
|
||||
|
||||
rustc.args(&self.props.compile_flags);
|
||||
|
@ -2509,7 +2521,6 @@ impl<'test> TestCx<'test> {
|
|||
.compile_flags
|
||||
.iter()
|
||||
.any(|s| s.contains("--error-format"));
|
||||
|
||||
let proc_res = self.compile_test();
|
||||
self.check_if_test_should_compile(&proc_res);
|
||||
|
||||
|
|
Loading…
Reference in New Issue