Rollup of 13 pull requests

Cargo.lock

@@ -404,9 +404,9 @@ version = "0.1.0"
 
 [[package]]
 name = "cc"
-version = "1.0.54"
+version = "1.0.57"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7bbb73db36c1246e9034e307d0fba23f9a2e251faa47ade70c1bd252220c8311"
+checksum = "0fde55d2a2bfaa4c9668bbc63f531fbdeee3ffe188f4662511ce2c22b3eedebe"
 dependencies = [
  "jobserver",
 ]
@@ -1366,8 +1366,8 @@ checksum = "7e81a7c05f79578dbc15793d8b619db9ba32b4577003ef3af1a91c416798c58d"
 name = "installer"
 version = "0.0.0"
 dependencies = [
+ "anyhow",
  "clap",
- "failure",
  "flate2",
  "lazy_static",
  "num_cpus",

src/bootstrap/native.rs

@@ -9,6 +9,7 @@
 //! ensure that they're always in place if needed.
 
 use std::env;
+use std::env::consts::EXE_EXTENSION;
 use std::ffi::OsString;
 use std::fs::{self, File};
 use std::io;
@@ -252,8 +253,14 @@ impl Step for Llvm {
  // FIXME: if the llvm root for the build triple is overridden then we
  // should use llvm-tblgen from there, also should verify that it
  // actually exists most of the time in normal installs of LLVM.
- let host = builder.llvm_out(builder.config.build).join("bin/llvm-tblgen");
- cfg.define("CMAKE_CROSSCOMPILING", "True").define("LLVM_TABLEGEN", &host);
+ let host_bin = builder.llvm_out(builder.config.build).join("bin");
+ cfg.define("CMAKE_CROSSCOMPILING", "True");
+ cfg.define("LLVM_TABLEGEN", host_bin.join("llvm-tblgen").with_extension(EXE_EXTENSION));
+ cfg.define("LLVM_NM", host_bin.join("llvm-nm").with_extension(EXE_EXTENSION));
+ cfg.define(
+ "LLVM_CONFIG_PATH",
+ host_bin.join("llvm-config").with_extension(EXE_EXTENSION),
+ );
 
  if target.contains("netbsd") {
  cfg.define("CMAKE_SYSTEM_NAME", "NetBSD");
@@ -262,8 +269,6 @@ impl Step for Llvm {
  } else if target.contains("windows") {
  cfg.define("CMAKE_SYSTEM_NAME", "Windows");
  }
-
- cfg.define("LLVM_NATIVE_BUILD", builder.llvm_out(builder.config.build).join("build"));
  }
 
  if let Some(ref suffix) = builder.config.llvm_version_suffix {
@@ -431,6 +436,9 @@ fn configure_cmake(
  cflags.push_str(" -miphoneos-version-min=10.0");
  }
  }
+ if builder.config.llvm_clang_cl.is_some() {
+ cflags.push_str(&format!(" --target={}", target))
+ }
  cfg.define("CMAKE_C_FLAGS", cflags);
  let mut cxxflags = builder.cflags(target, GitRepo::Llvm).join(" ");
  if builder.config.llvm_static_stdcpp && !target.contains("msvc") && !target.contains("netbsd") {
@@ -439,6 +447,9 @@ fn configure_cmake(
  if let Some(ref s) = builder.config.llvm_cxxflags {
  cxxflags.push_str(&format!(" {}", s));
  }
+ if builder.config.llvm_clang_cl.is_some() {
+ cxxflags.push_str(&format!(" --target={}", target))
+ }
  cfg.define("CMAKE_CXX_FLAGS", cxxflags);
  if let Some(ar) = builder.ar(target) {
  if ar.is_absolute() {
@@ -484,7 +495,7 @@ impl Step for Lld {
  run.builder.ensure(Lld { target: run.target });
  }
 
- /// Compile LLVM for `target`.
+ /// Compile LLD for `target`.
  fn run(self, builder: &Builder<'_>) -> PathBuf {
  if builder.config.dry_run {
  return PathBuf::from("lld-out-dir-test-gen");
@@ -521,6 +532,7 @@ impl Step for Lld {
  // can't build on a system where your paths require `\` on Windows, but
  // there's probably a lot of reasons you can't do that other than this.
  let llvm_config_shim = env::current_exe().unwrap().with_file_name("llvm-config-wrapper");
+
  cfg.out_dir(&out_dir)
  .profile("Release")
  .env("LLVM_CONFIG_REAL", &llvm_config)
@@ -543,7 +555,10 @@ impl Step for Lld {
  if target != builder.config.build {
  cfg.env("LLVM_CONFIG_SHIM_REPLACE", &builder.config.build)
  .env("LLVM_CONFIG_SHIM_REPLACE_WITH", &target)
- .define("LLVM_TABLEGEN_EXE", llvm_config.with_file_name("llvm-tblgen"));
+ .define(
+ "LLVM_TABLEGEN_EXE",
+ llvm_config.with_file_name("llvm-tblgen").with_extension(EXE_EXTENSION),
+ );
  }
 
  // Explicitly set C++ standard, because upstream doesn't do so
@@ -595,8 +610,8 @@ impl Step for TestHelpers {
  }
 
  // We may have found various cross-compilers a little differently due to our
- // extra configuration, so inform gcc of these compilers. Note, though, that
- // on MSVC we still need gcc's detection of env vars (ugh).
+ // extra configuration, so inform cc of these compilers. Note, though, that
+ // on MSVC we still need cc's detection of env vars (ugh).
  if !target.contains("msvc") {
  if let Some(ar) = builder.ar(target) {
  cfg.archiver(ar);

src/bootstrap/test.rs

@@ -1562,7 +1562,7 @@ impl Step for CrateLibrustc {
  let compiler = builder.compiler(builder.top_stage, run.host);
 
  for krate in builder.in_tree_crates("rustc-main") {
- if run.path.ends_with(&krate.path) {
+ if krate.path.ends_with(&run.path) {
  let test_kind = builder.kind.into();
 
  builder.ensure(CrateLibrustc {
@@ -1669,7 +1669,7 @@ impl Step for Crate {
  };
 
  for krate in builder.in_tree_crates("test") {
- if run.path.ends_with(&krate.local_path(&builder)) {
+ if krate.path.ends_with(&run.path) {
  make(Mode::Std, krate);
  }
  }

src/doc/rustdoc/src/unstable-features.md

@@ -321,7 +321,7 @@ library, as an equivalent command-line argument is provided to `rustc` when buil
 ### `--index-page`: provide a top-level landing page for docs
 
 This feature allows you to generate an index-page with a given markdown file. A good example of it
-is the [rust documentation index](https://doc.rust-lang.org/index.html).
+is the [rust documentation index](https://doc.rust-lang.org/nightly/index.html).
 
 With this, you'll have a page which you can custom as much as you want at the top of your crates.
 

src/liballoc/slice.rs

@@ -136,8 +136,6 @@ pub use hack::to_vec;
 // `test_permutations` test
 mod hack {
  use crate::boxed::Box;
- #[cfg(test)]
- use crate::string::ToString;
  use crate::vec::Vec;
 
  // We shouldn't add inline attribute to this since this is used in
@@ -156,9 +154,9 @@ mod hack {
  where
  T: Clone,
  {
- let mut vector = Vec::with_capacity(s.len());
- vector.extend_from_slice(s);
- vector
+ let mut vec = Vec::with_capacity(s.len());
+ vec.extend_from_slice(s);
+ vec
  }
 }
 

src/librustc_error_codes/error_codes/E0570.md

@@ -1,6 +1,6 @@
 The requested ABI is unsupported by the current target.
 
-The rust compiler maintains for each target a blacklist of ABIs unsupported on
+The rust compiler maintains for each target a list of unsupported ABIs on
 that target. If an ABI is present in such a list this usually means that the
 target / ABI combination is currently unsupported by llvm.
 

src/librustc_errors/json/tests.rs

@@ -59,6 +59,7 @@ fn test_positions(code: &str, span: (u32, u32), expected_output: SpanTestData) {
  sm,
  true,
  HumanReadableErrorType::Short(ColorConfig::Never),
+ None,
  false,
  );
 

src/librustc_lint/lib.rs

@@ -63,7 +63,7 @@ use rustc_middle::ty::query::Providers;
 use rustc_middle::ty::TyCtxt;
 use rustc_session::lint::builtin::{
  BARE_TRAIT_OBJECTS, ELIDED_LIFETIMES_IN_PATHS, EXPLICIT_OUTLIVES_REQUIREMENTS,
- INTRA_DOC_LINK_RESOLUTION_FAILURE, INVALID_CODEBLOCK_ATTRIBUTE, MISSING_DOC_CODE_EXAMPLES,
+ INTRA_DOC_LINK_RESOLUTION_FAILURE, INVALID_CODEBLOCK_ATTRIBUTES, MISSING_DOC_CODE_EXAMPLES,
  PRIVATE_DOC_TESTS,
 };
 use rustc_span::symbol::{Ident, Symbol};
@@ -305,7 +305,7 @@ fn register_builtins(store: &mut LintStore, no_interleave_lints: bool) {
  add_lint_group!(
  "rustdoc",
  INTRA_DOC_LINK_RESOLUTION_FAILURE,
- INVALID_CODEBLOCK_ATTRIBUTE,
+ INVALID_CODEBLOCK_ATTRIBUTES,
  MISSING_DOC_CODE_EXAMPLES,
  PRIVATE_DOC_TESTS
  );

src/librustc_mir_build/build/mod.rs

@@ -10,6 +10,7 @@ use rustc_hir::lang_items;
 use rustc_hir::{GeneratorKind, HirIdMap, Node};
 use rustc_index::vec::{Idx, IndexVec};
 use rustc_infer::infer::TyCtxtInferExt;
+use rustc_middle::middle::codegen_fn_attrs::CodegenFnAttrFlags;
 use rustc_middle::middle::region;
 use rustc_middle::mir::*;
 use rustc_middle::ty::subst::Subst;
@@ -790,12 +791,22 @@ impl<'a, 'tcx> Builder<'a, 'tcx> {
  argument_scope: region::Scope,
  ast_body: &'tcx hir::Expr<'tcx>,
  ) -> BlockAnd<()> {
+ let tcx = self.hir.tcx();
+ let attrs = tcx.codegen_fn_attrs(fn_def_id);
+ let naked = attrs.flags.contains(CodegenFnAttrFlags::NAKED);
+
  // Allocate locals for the function arguments
  for &ArgInfo(ty, _, arg_opt, _) in arguments.iter() {
  let source_info =
  SourceInfo::outermost(arg_opt.map_or(self.fn_span, |arg| arg.pat.span));
  let arg_local = self.local_decls.push(LocalDecl::with_source_info(ty, source_info));
 
+ // Emit function argument debuginfo only for non-naked functions.
+ // See: https://github.com/rust-lang/rust/issues/42779
+ if naked {
+ continue;
+ }
+
  // If this is a simple binding pattern, give debuginfo a nice name.
  if let Some(arg) = arg_opt {
  if let Some(ident) = arg.pat.simple_ident() {
@@ -808,7 +819,6 @@ impl<'a, 'tcx> Builder<'a, 'tcx> {
  }
  }
 
- let tcx = self.hir.tcx();
  let tcx_hir = tcx.hir();
  let hir_tables = self.hir.tables();
 

src/librustc_parse/parser/expr.rs

@@ -1790,7 +1790,7 @@ impl<'a> Parser<'a> {
  let require_comma = classify::expr_requires_semi_to_be_stmt(&expr)
  && self.token != token::CloseDelim(token::Brace);
 
- let hi = self.token.span;
+ let hi = self.prev_token.span;
 
  if require_comma {
  let sm = self.sess.source_map();

src/librustc_passes/check_attr.rs

@@ -292,6 +292,8 @@ impl CheckAttrVisitor<'tcx> {
  | sym::u32
  | sym::i64
  | sym::u64
+ | sym::i128
+ | sym::u128
  | sym::isize
  | sym::usize => {
  int_reprs += 1;

src/librustc_resolve/imports.rs

@@ -262,8 +262,8 @@ impl<'a> Resolver<'a> {
  }
 
  let check_usable = |this: &mut Self, binding: &'a NameBinding<'a>| {
- if let Some(blacklisted_binding) = this.blacklisted_binding {
- if ptr::eq(binding, blacklisted_binding) {
+ if let Some(unusable_binding) = this.unusable_binding {
+ if ptr::eq(binding, unusable_binding) {
  return Err((Determined, Weak::No));
  }
  }
@@ -278,12 +278,12 @@ impl<'a> Resolver<'a> {
  return resolution
  .binding
  .and_then(|binding| {
- // If the primary binding is blacklisted, search further and return the shadowed
- // glob binding if it exists. What we really want here is having two separate
- // scopes in a module - one for non-globs and one for globs, but until that's done
- // use this hack to avoid inconsistent resolution ICEs during import validation.
- if let Some(blacklisted_binding) = self.blacklisted_binding {
- if ptr::eq(binding, blacklisted_binding) {
+ // If the primary binding is unusable, search further and return the shadowed glob
+ // binding if it exists. What we really want here is having two separate scopes in
+ // a module - one for non-globs and one for globs, but until that's done use this
+ // hack to avoid inconsistent resolution ICEs during import validation.
+ if let Some(unusable_binding) = self.unusable_binding {
+ if ptr::eq(binding, unusable_binding) {
  return resolution.shadowed_glob;
  }
  }
@@ -875,9 +875,9 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  /// consolidate multiple unresolved import errors into a single diagnostic.
  fn finalize_import(&mut self, import: &'b Import<'b>) -> Option<UnresolvedImportError> {
  let orig_vis = import.vis.replace(ty::Visibility::Invisible);
- let orig_blacklisted_binding = match &import.kind {
+ let orig_unusable_binding = match &import.kind {
  ImportKind::Single { target_bindings, .. } => {
- Some(mem::replace(&mut self.r.blacklisted_binding, target_bindings[TypeNS].get()))
+ Some(mem::replace(&mut self.r.unusable_binding, target_bindings[TypeNS].get()))
  }
  _ => None,
  };
@@ -891,8 +891,8 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  import.crate_lint(),
  );
  let no_ambiguity = self.r.ambiguity_errors.len() == prev_ambiguity_errors_len;
- if let Some(orig_blacklisted_binding) = orig_blacklisted_binding {
- self.r.blacklisted_binding = orig_blacklisted_binding;
+ if let Some(orig_unusable_binding) = orig_unusable_binding {
+ self.r.unusable_binding = orig_unusable_binding;
  }
  import.vis.set(orig_vis);
  if let PathResult::Failed { .. } | PathResult::NonModule(..) = path_res {
@@ -1013,8 +1013,8 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  self.r.per_ns(|this, ns| {
  if !type_ns_only || ns == TypeNS {
  let orig_vis = import.vis.replace(ty::Visibility::Invisible);
- let orig_blacklisted_binding =
- mem::replace(&mut this.blacklisted_binding, target_bindings[ns].get());
+ let orig_unusable_binding =
+ mem::replace(&mut this.unusable_binding, target_bindings[ns].get());
  let orig_last_import_segment = mem::replace(&mut this.last_import_segment, true);
  let binding = this.resolve_ident_in_module(
  module,
@@ -1025,7 +1025,7 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  import.span,
  );
  this.last_import_segment = orig_last_import_segment;
- this.blacklisted_binding = orig_blacklisted_binding;
+ this.unusable_binding = orig_unusable_binding;
  import.vis.set(orig_vis);
 
  match binding {
@@ -1291,8 +1291,8 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  return;
  }
 
- let orig_blacklisted_binding =
- mem::replace(&mut this.blacklisted_binding, target_bindings[ns].get());
+ let orig_unusable_binding =
+ mem::replace(&mut this.unusable_binding, target_bindings[ns].get());
 
  match this.early_resolve_ident_in_lexical_scope(
  target,
@@ -1311,7 +1311,7 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
  Err(_) => is_redundant[ns] = Some(false),
  }
 
- this.blacklisted_binding = orig_blacklisted_binding;
+ this.unusable_binding = orig_unusable_binding;
  }
  });
 

src/librustc_resolve/late.rs

@@ -842,14 +842,14 @@ impl<'a, 'b, 'ast> LateResolutionVisitor<'a, 'b, 'ast> {
  report_error(self, ns);
  }
  Some(LexicalScopeBinding::Item(binding)) => {
- let orig_blacklisted_binding =
- replace(&mut self.r.blacklisted_binding, Some(binding));
+ let orig_unusable_binding =
+ replace(&mut self.r.unusable_binding, Some(binding));
  if let Some(LexicalScopeBinding::Res(..)) = self
  .resolve_ident_in_lexical_scope(ident, ns, None, use_tree.prefix.span)
  {
  report_error(self, ns);
  }
- self.r.blacklisted_binding = orig_blacklisted_binding;
+ self.r.unusable_binding = orig_unusable_binding;
  }
  None => {}
  }

src/librustc_resolve/lib.rs

@@ -867,7 +867,7 @@ pub struct Resolver<'a> {
  last_import_segment: bool,
  /// This binding should be ignored during in-module resolution, so that we don't get
  /// "self-confirming" import resolutions during import validation.
- blacklisted_binding: Option<&'a NameBinding<'a>>,
+ unusable_binding: Option<&'a NameBinding<'a>>,
 
  /// The idents for the primitive types.
  primitive_type_table: PrimitiveTypeTable,
@@ -1266,7 +1266,7 @@ impl<'a> Resolver<'a> {
  indeterminate_imports: Vec::new(),
 
  last_import_segment: false,
- blacklisted_binding: None,
+ unusable_binding: None,
 
  primitive_type_table: PrimitiveTypeTable::new(),