src/librustdoc/core.rs

   1 use rustc_ast as ast;
   2 use rustc_data_structures::fx::{FxHashMap, FxHashSet};
   3 use rustc_data_structures::sync::{self, Lrc};
   4 use rustc_driver::abort_on_err;
   5 use rustc_errors::emitter::{Emitter, EmitterWriter};
   6 use rustc_errors::json::JsonEmitter;
   7 use rustc_feature::UnstableFeatures;
   8 use rustc_hir::def::Res;
   9 use rustc_hir::def_id::{DefId, LocalDefId};
  10 use rustc_hir::HirId;
  11 use rustc_hir::{
  12     intravisit::{self, NestedVisitorMap, Visitor},
  13     Path,
  14 };
  15 use rustc_interface::{interface, Queries};
  16 use rustc_middle::hir::map::Map;
  17 use rustc_middle::middle::privacy::AccessLevels;
  18 use rustc_middle::ty::{ParamEnv, Ty, TyCtxt};
  19 use rustc_resolve as resolve;
  20 use rustc_session::config::{self, CrateType, ErrorOutputType};
  21 use rustc_session::lint;
  22 use rustc_session::DiagnosticOutput;
  23 use rustc_session::Session;
  24 use rustc_span::source_map;
  25 use rustc_span::symbol::sym;
  26 use rustc_span::Span;
  27
  28 use std::cell::RefCell;
  29 use std::mem;
  30 use std::rc::Rc;
  31
  32 use crate::clean::inline::build_external_trait;
  33 use crate::clean::{self, FakeDefId, TraitWithExtraInfo};
  34 use crate::config::{Options as RustdocOptions, OutputFormat, RenderOptions};
  35 use crate::formats::cache::Cache;
  36 use crate::passes::{self, Condition::*, ConditionalPass};
  37
  38 crate use rustc_session::config::{DebuggingOptions, Input, Options};
  39
  40 crate struct DocContext<'tcx> {
  41     crate tcx: TyCtxt<'tcx>,
  42     /// Name resolver. Used for intra-doc links.
  43     ///
  44     /// The `Rc<RefCell<...>>` wrapping is needed because that is what's returned by
  45     /// [`Queries::expansion()`].
  46     // FIXME: see if we can get rid of this RefCell somehow
  47     crate resolver: Rc<RefCell<interface::BoxedResolver>>,
  48     /// Used for normalization.
  49     ///
  50     /// Most of this logic is copied from rustc_lint::late.
  51     crate param_env: ParamEnv<'tcx>,
  52     /// Later on moved through `clean::Crate` into `cache`
  53     crate external_traits: Rc<RefCell<FxHashMap<DefId, clean::TraitWithExtraInfo>>>,
  54     /// Used while populating `external_traits` to ensure we don't process the same trait twice at
  55     /// the same time.
  56     crate active_extern_traits: FxHashSet<DefId>,
  57     // The current set of type and lifetime substitutions,
  58     // for expanding type aliases at the HIR level:
  59     /// Table `DefId` of type parameter -> substituted type
  60     crate ty_substs: FxHashMap<DefId, clean::Type>,
  61     /// Table `DefId` of lifetime parameter -> substituted lifetime
  62     crate lt_substs: FxHashMap<DefId, clean::Lifetime>,
  63     /// Table `DefId` of const parameter -> substituted const
  64     crate ct_substs: FxHashMap<DefId, clean::Constant>,
  65     /// Table synthetic type parameter for `impl Trait` in argument position -> bounds
  66     crate impl_trait_bounds: FxHashMap<ImplTraitParam, Vec<clean::GenericBound>>,
  67     /// Auto-trait or blanket impls processed so far, as `(self_ty, trait_def_id)`.
  68     // FIXME(eddyb) make this a `ty::TraitRef<'tcx>` set.
  69     crate generated_synthetics: FxHashSet<(Ty<'tcx>, DefId)>,
  70     crate auto_traits: Vec<DefId>,
  71     /// The options given to rustdoc that could be relevant to a pass.
  72     crate render_options: RenderOptions,
  73     /// The traits in scope for a given module.
  74     ///
  75     /// See `collect_intra_doc_links::traits_implemented_by` for more details.
  76     /// `map<module, set<trait>>`
  77     crate module_trait_cache: FxHashMap<DefId, FxHashSet<DefId>>,
  78     /// This same cache is used throughout rustdoc, including in [`crate::html::render`].
  79     crate cache: Cache,
  80     /// Used by [`clean::inline`] to tell if an item has already been inlined.
  81     crate inlined: FxHashSet<FakeDefId>,
  82     /// Used by `calculate_doc_coverage`.
  83     crate output_format: OutputFormat,
  84 }
  85
  86 impl<'tcx> DocContext<'tcx> {
  87     crate fn sess(&self) -> &'tcx Session {
  88         &self.tcx.sess
  89     }
  90
  91     crate fn with_param_env<T, F: FnOnce(&mut Self) -> T>(&mut self, def_id: DefId, f: F) -> T {
  92         let old_param_env = mem::replace(&mut self.param_env, self.tcx.param_env(def_id));
  93         let ret = f(self);
  94         self.param_env = old_param_env;
  95         ret
  96     }
  97
  98     crate fn enter_resolver<F, R>(&self, f: F) -> R
  99     where
 100         F: FnOnce(&mut resolve::Resolver<'_>) -> R,
 101     {
 102         self.resolver.borrow_mut().access(f)
 103     }
 104
 105     /// Call the closure with the given parameters set as
 106     /// the substitutions for a type alias' RHS.
 107     crate fn enter_alias<F, R>(
 108         &mut self,
 109         ty_substs: FxHashMap<DefId, clean::Type>,
 110         lt_substs: FxHashMap<DefId, clean::Lifetime>,
 111         ct_substs: FxHashMap<DefId, clean::Constant>,
 112         f: F,
 113     ) -> R
 114     where
 115         F: FnOnce(&mut Self) -> R,
 116     {
 117         let (old_tys, old_lts, old_cts) = (
 118             mem::replace(&mut self.ty_substs, ty_substs),
 119             mem::replace(&mut self.lt_substs, lt_substs),
 120             mem::replace(&mut self.ct_substs, ct_substs),
 121         );
 122         let r = f(self);
 123         self.ty_substs = old_tys;
 124         self.lt_substs = old_lts;
 125         self.ct_substs = old_cts;
 126         r
 127     }
 128
 129     /// Like `hir().local_def_id_to_hir_id()`, but skips calling it on fake DefIds.
 130     /// (This avoids a slice-index-out-of-bounds panic.)
 131     crate fn as_local_hir_id(tcx: TyCtxt<'_>, def_id: FakeDefId) -> Option<HirId> {
 132         match def_id {
 133             FakeDefId::Real(real_id) => {
 134                 real_id.as_local().map(|def_id| tcx.hir().local_def_id_to_hir_id(def_id))
 135             }
 136             FakeDefId::Fake(_, _) => None,
 137         }
 138     }
 139 }
 140
 141 /// Creates a new diagnostic `Handler` that can be used to emit warnings and errors.
 142 ///
 143 /// If the given `error_format` is `ErrorOutputType::Json` and no `SourceMap` is given, a new one
 144 /// will be created for the handler.
 145 crate fn new_handler(
 146     error_format: ErrorOutputType,
 147     source_map: Option<Lrc<source_map::SourceMap>>,
 148     debugging_opts: &DebuggingOptions,
 149 ) -> rustc_errors::Handler {
 150     let emitter: Box<dyn Emitter + sync::Send> = match error_format {
 151         ErrorOutputType::HumanReadable(kind) => {
 152             let (short, color_config) = kind.unzip();
 153             Box::new(
 154                 EmitterWriter::stderr(
 155                     color_config,
 156                     source_map.map(|sm| sm as _),
 157                     short,
 158                     debugging_opts.teach,
 159                     debugging_opts.terminal_width,
 160                     false,
 161                 )
 162                 .ui_testing(debugging_opts.ui_testing),
 163             )
 164         }
 165         ErrorOutputType::Json { pretty, json_rendered } => {
 166             let source_map = source_map.unwrap_or_else(|| {
 167                 Lrc::new(source_map::SourceMap::new(source_map::FilePathMapping::empty()))
 168             });
 169             Box::new(
 170                 JsonEmitter::stderr(
 171                     None,
 172                     source_map,
 173                     pretty,
 174                     json_rendered,
 175                     debugging_opts.terminal_width,
 176                     false,
 177                 )
 178                 .ui_testing(debugging_opts.ui_testing),
 179             )
 180         }
 181     };
 182
 183     rustc_errors::Handler::with_emitter_and_flags(
 184         emitter,
 185         debugging_opts.diagnostic_handler_flags(true),
 186     )
 187 }
 188
 189 /// Parse, resolve, and typecheck the given crate.
 190 crate fn create_config(
 191     RustdocOptions {
 192         input,
 193         crate_name,
 194         proc_macro_crate,
 195         error_format,
 196         libs,
 197         externs,
 198         mut cfgs,
 199         codegen_options,
 200         debugging_opts,
 201         target,
 202         edition,
 203         maybe_sysroot,
 204         lint_opts,
 205         describe_lints,
 206         lint_cap,
 207         display_warnings,
 208         ..
 209     }: RustdocOptions,
 210 ) -> rustc_interface::Config {
 211     // Add the doc cfg into the doc build.
 212     cfgs.push("doc".to_string());
 213
 214     let cpath = Some(input.clone());
 215     let input = Input::File(input);
 216
 217     // By default, rustdoc ignores all lints.
 218     // Specifically unblock lints relevant to documentation or the lint machinery itself.
 219     let mut lints_to_show = vec![
 220         // it's unclear whether these should be part of rustdoc directly (#77364)
 221         rustc_lint::builtin::MISSING_DOCS.name.to_string(),
 222         rustc_lint::builtin::INVALID_DOC_ATTRIBUTES.name.to_string(),
 223         // these are definitely not part of rustdoc, but we want to warn on them anyway.
 224         rustc_lint::builtin::RENAMED_AND_REMOVED_LINTS.name.to_string(),
 225         rustc_lint::builtin::UNKNOWN_LINTS.name.to_string(),
 226     ];
 227     lints_to_show.extend(crate::lint::RUSTDOC_LINTS.iter().map(|lint| lint.name.to_string()));
 228
 229     let (lint_opts, lint_caps) = crate::lint::init_lints(lints_to_show, lint_opts, |lint| {
 230         Some((lint.name_lower(), lint::Allow))
 231     });
 232
 233     let crate_types =
 234         if proc_macro_crate { vec![CrateType::ProcMacro] } else { vec![CrateType::Rlib] };
 235     // plays with error output here!
 236     let sessopts = config::Options {
 237         maybe_sysroot,
 238         search_paths: libs,
 239         crate_types,
 240         lint_opts: if !display_warnings { lint_opts } else { vec![] },
 241         lint_cap,
 242         cg: codegen_options,
 243         externs,
 244         target_triple: target,
 245         unstable_features: UnstableFeatures::from_environment(crate_name.as_deref()),
 246         actually_rustdoc: true,
 247         debugging_opts,
 248         error_format,
 249         edition,
 250         describe_lints,
 251         crate_name,
 252         ..Options::default()
 253     };
 254
 255     interface::Config {
 256         opts: sessopts,
 257         crate_cfg: interface::parse_cfgspecs(cfgs),
 258         input,
 259         input_path: cpath,
 260         output_file: None,
 261         output_dir: None,
 262         file_loader: None,
 263         diagnostic_output: DiagnosticOutput::Default,
 264         stderr: None,
 265         lint_caps,
 266         parse_sess_created: None,
 267         register_lints: Some(box crate::lint::register_lints),
 268         override_queries: Some(|_sess, providers, _external_providers| {
 269             // Most lints will require typechecking, so just don't run them.
 270             providers.lint_mod = |_, _| {};
 271             // Prevent `rustc_typeck::check_crate` from calling `typeck` on all bodies.
 272             providers.typeck_item_bodies = |_, _| {};
 273             // hack so that `used_trait_imports` won't try to call typeck
 274             providers.used_trait_imports = |_, _| {
 275                 lazy_static! {
 276                     static ref EMPTY_SET: FxHashSet<LocalDefId> = FxHashSet::default();
 277                 }
 278                 &EMPTY_SET
 279             };
 280             // In case typeck does end up being called, don't ICE in case there were name resolution errors
 281             providers.typeck = move |tcx, def_id| {
 282                 // Closures' tables come from their outermost function,
 283                 // as they are part of the same "inference environment".
 284                 // This avoids emitting errors for the parent twice (see similar code in `typeck_with_fallback`)
 285                 let outer_def_id = tcx.closure_base_def_id(def_id.to_def_id()).expect_local();
 286                 if outer_def_id != def_id {
 287                     return tcx.typeck(outer_def_id);
 288                 }
 289
 290                 let hir = tcx.hir();
 291                 let body = hir.body(hir.body_owned_by(hir.local_def_id_to_hir_id(def_id)));
 292                 debug!("visiting body for {:?}", def_id);
 293                 EmitIgnoredResolutionErrors::new(tcx).visit_body(body);
 294                 (rustc_interface::DEFAULT_QUERY_PROVIDERS.typeck)(tcx, def_id)
 295             };
 296         }),
 297         make_codegen_backend: None,
 298         registry: rustc_driver::diagnostics_registry(),
 299     }
 300 }
 301
 302 crate fn create_resolver<'a>(
 303     queries: &Queries<'a>,
 304     sess: &Session,
 305 ) -> Rc<RefCell<interface::BoxedResolver>> {
 306     let parts = abort_on_err(queries.expansion(), sess).peek();
 307     let (krate, resolver, _) = &*parts;
 308     let resolver = resolver.borrow().clone();
 309
 310     let mut loader = crate::passes::collect_intra_doc_links::IntraLinkCrateLoader::new(resolver);
 311     ast::visit::walk_crate(&mut loader, krate);
 312
 313     loader.resolver
 314 }
 315
 316 crate fn run_global_ctxt(
 317     tcx: TyCtxt<'_>,
 318     resolver: Rc<RefCell<interface::BoxedResolver>>,
 319     mut default_passes: passes::DefaultPassOption,
 320     manual_passes: Vec<String>,
 321     render_options: RenderOptions,
 322     output_format: OutputFormat,
 323 ) -> (clean::Crate, RenderOptions, Cache) {
 324     // Certain queries assume that some checks were run elsewhere
 325     // (see https://github.com/rust-lang/rust/pull/73566#issuecomment-656954425),
 326     // so type-check everything other than function bodies in this crate before running lints.
 327
 328     // NOTE: this does not call `tcx.analysis()` so that we won't
 329     // typeck function bodies or run the default rustc lints.
 330     // (see `override_queries` in the `config`)
 331
 332     // HACK(jynelson) this calls an _extremely_ limited subset of `typeck`
 333     // and might break if queries change their assumptions in the future.
 334
 335     // NOTE: This is copy/pasted from typeck/lib.rs and should be kept in sync with those changes.
 336     tcx.sess.time("item_types_checking", || {
 337         for &module in tcx.hir().krate().modules.keys() {
 338             tcx.ensure().check_mod_item_types(module);
 339         }
 340     });
 341     tcx.sess.abort_if_errors();
 342     tcx.sess.time("missing_docs", || {
 343         rustc_lint::check_crate(tcx, rustc_lint::builtin::MissingDoc::new);
 344     });
 345     tcx.sess.time("check_mod_attrs", || {
 346         for &module in tcx.hir().krate().modules.keys() {
 347             tcx.ensure().check_mod_attrs(module);
 348         }
 349     });
 350     rustc_passes::stability::check_unused_or_stable_features(tcx);
 351
 352     let access_levels = tcx.privacy_access_levels(());
 353     // Convert from a HirId set to a DefId set since we don't always have easy access
 354     // to the map from defid -> hirid
 355     let access_levels = AccessLevels {
 356         map: access_levels
 357             .map
 358             .iter()
 359             .map(|(&k, &v)| (tcx.hir().local_def_id(k).to_def_id(), v))
 360             .collect(),
 361     };
 362
 363     let mut ctxt = DocContext {
 364         tcx,
 365         resolver,
 366         param_env: ParamEnv::empty(),
 367         external_traits: Default::default(),
 368         active_extern_traits: Default::default(),
 369         ty_substs: Default::default(),
 370         lt_substs: Default::default(),
 371         ct_substs: Default::default(),
 372         impl_trait_bounds: Default::default(),
 373         generated_synthetics: Default::default(),
 374         auto_traits: tcx
 375             .all_traits(())
 376             .iter()
 377             .cloned()
 378             .filter(|trait_def_id| tcx.trait_is_auto(*trait_def_id))
 379             .collect(),
 380         module_trait_cache: FxHashMap::default(),
 381         cache: Cache::new(access_levels, render_options.document_private),
 382         inlined: FxHashSet::default(),
 383         output_format,
 384         render_options,
 385     };
 386
 387     // Small hack to force the Sized trait to be present.
 388     //
 389     // Note that in case of `#![no_core]`, the trait is not available.
 390     if let Some(sized_trait_did) = ctxt.tcx.lang_items().sized_trait() {
 391         let mut sized_trait = build_external_trait(&mut ctxt, sized_trait_did);
 392         sized_trait.is_auto = true;
 393         ctxt.external_traits
 394             .borrow_mut()
 395             .insert(sized_trait_did, TraitWithExtraInfo { trait_: sized_trait, is_notable: false });
 396     }
 397
 398     debug!("crate: {:?}", tcx.hir().krate());
 399
 400     let mut krate = tcx.sess.time("clean_crate", || clean::krate(&mut ctxt));
 401
 402     if krate.module.doc_value().map(|d| d.is_empty()).unwrap_or(true) {
 403         let help = format!(
 404             "The following guide may be of use:\n\
 405             {}/rustdoc/how-to-write-documentation.html",
 406             crate::DOC_RUST_LANG_ORG_CHANNEL
 407         );
 408         tcx.struct_lint_node(
 409             crate::lint::MISSING_CRATE_LEVEL_DOCS,
 410             DocContext::as_local_hir_id(tcx, krate.module.def_id).unwrap(),
 411             |lint| {
 412                 let mut diag =
 413                     lint.build("no documentation found for this crate's top-level module");
 414                 diag.help(&help);
 415                 diag.emit();
 416             },
 417         );
 418     }
 419
 420     fn report_deprecated_attr(name: &str, diag: &rustc_errors::Handler, sp: Span) {
 421         let mut msg =
 422             diag.struct_span_warn(sp, &format!("the `#![doc({})]` attribute is deprecated", name));
 423         msg.note(
 424             "see issue #44136 <https://github.com/rust-lang/rust/issues/44136> \
 425              for more information",
 426         );
 427
 428         if name == "no_default_passes" {
 429             msg.help("you may want to use `#![doc(document_private_items)]`");
 430         } else if name.starts_with("plugins") {
 431             msg.warn("`#![doc(plugins = \"...\")]` no longer functions; see CVE-2018-1000622 <https://nvd.nist.gov/vuln/detail/CVE-2018-1000622>");
 432         }
 433
 434         msg.emit();
 435     }
 436
 437     let parse_pass = |name: &str, sp: Option<Span>| {
 438         if let Some(pass) = passes::find_pass(name) {
 439             Some(ConditionalPass::always(pass))
 440         } else {
 441             let msg = &format!("ignoring unknown pass `{}`", name);
 442             let mut warning = if let Some(sp) = sp {
 443                 tcx.sess.struct_span_warn(sp, msg)
 444             } else {
 445                 tcx.sess.struct_warn(msg)
 446             };
 447             if name == "collapse-docs" {
 448                 warning.note("the `collapse-docs` pass was removed in #80261 <https://github.com/rust-lang/rust/pull/80261>");
 449             }
 450             warning.emit();
 451             None
 452         }
 453     };
 454
 455     let mut manual_passes: Vec<_> =
 456         manual_passes.into_iter().flat_map(|name| parse_pass(&name, None)).collect();
 457
 458     // Process all of the crate attributes, extracting plugin metadata along
 459     // with the passes which we are supposed to run.
 460     for attr in krate.module.attrs.lists(sym::doc) {
 461         let diag = ctxt.sess().diagnostic();
 462
 463         let name = attr.name_or_empty();
 464         if attr.is_word() {
 465             if name == sym::no_default_passes {
 466                 report_deprecated_attr("no_default_passes", diag, attr.span());
 467                 if default_passes == passes::DefaultPassOption::Default {
 468                     default_passes = passes::DefaultPassOption::None;
 469                 }
 470             }
 471         } else if let Some(value) = attr.value_str() {
 472             match name {
 473                 sym::passes => {
 474                     report_deprecated_attr("passes = \"...\"", diag, attr.span());
 475                 }
 476                 sym::plugins => {
 477                     report_deprecated_attr("plugins = \"...\"", diag, attr.span());
 478                     continue;
 479                 }
 480                 _ => continue,
 481             };
 482             for name in value.as_str().split_whitespace() {
 483                 let span = attr.name_value_literal_span().unwrap_or(attr.span());
 484                 manual_passes.extend(parse_pass(name, Some(span)));
 485             }
 486         }
 487
 488         if attr.is_word() && name == sym::document_private_items {
 489             ctxt.render_options.document_private = true;
 490         }
 491     }
 492
 493     let passes = passes::defaults(default_passes).iter().copied().chain(manual_passes);
 494     info!("Executing passes");
 495
 496     for p in passes {
 497         let run = match p.condition {
 498             Always => true,
 499             WhenDocumentPrivate => ctxt.render_options.document_private,
 500             WhenNotDocumentPrivate => !ctxt.render_options.document_private,
 501             WhenNotDocumentHidden => !ctxt.render_options.document_hidden,
 502         };
 503         if run {
 504             debug!("running pass {}", p.pass.name);
 505             krate = ctxt.tcx.sess.time(p.pass.name, || (p.pass.run)(krate, &mut ctxt));
 506         }
 507     }
 508
 509     ctxt.sess().abort_if_errors();
 510
 511     let render_options = ctxt.render_options;
 512     let mut cache = ctxt.cache;
 513     krate = tcx.sess.time("create_format_cache", || {
 514         cache.populate(krate, tcx, &render_options.extern_html_root_urls, &render_options.output)
 515     });
 516
 517     // The main crate doc comments are always collapsed.
 518     krate.collapsed = true;
 519
 520     (krate, render_options, cache)
 521 }
 522
 523 /// Due to <https://github.com/rust-lang/rust/pull/73566>,
 524 /// the name resolution pass may find errors that are never emitted.
 525 /// If typeck is called after this happens, then we'll get an ICE:
 526 /// 'Res::Error found but not reported'. To avoid this, emit the errors now.
 527 struct EmitIgnoredResolutionErrors<'tcx> {
 528     tcx: TyCtxt<'tcx>,
 529 }
 530
 531 impl<'tcx> EmitIgnoredResolutionErrors<'tcx> {
 532     fn new(tcx: TyCtxt<'tcx>) -> Self {
 533         Self { tcx }
 534     }
 535 }
 536
 537 impl<'tcx> Visitor<'tcx> for EmitIgnoredResolutionErrors<'tcx> {
 538     type Map = Map<'tcx>;
 539
 540     fn nested_visit_map(&mut self) -> NestedVisitorMap<Self::Map> {
 541         // We need to recurse into nested closures,
 542         // since those will fallback to the parent for type checking.
 543         NestedVisitorMap::OnlyBodies(self.tcx.hir())
 544     }
 545
 546     fn visit_path(&mut self, path: &'tcx Path<'_>, _id: HirId) {
 547         debug!("visiting path {:?}", path);
 548         if path.res == Res::Err {
 549             // We have less context here than in rustc_resolve,
 550             // so we can only emit the name and span.
 551             // However we can give a hint that rustc_resolve will have more info.
 552             let label = format!(
 553                 "could not resolve path `{}`",
 554                 path.segments
 555                     .iter()
 556                     .map(|segment| segment.ident.as_str().to_string())
 557                     .collect::<Vec<_>>()
 558                     .join("::")
 559             );
 560             let mut err = rustc_errors::struct_span_err!(
 561                 self.tcx.sess,
 562                 path.span,
 563                 E0433,
 564                 "failed to resolve: {}",
 565                 label
 566             );
 567             err.span_label(path.span, label);
 568             err.note("this error was originally ignored because you are running `rustdoc`");
 569             err.note("try running again with `rustc` or `cargo check` and you may get a more detailed error");
 570             err.emit();
 571         }
 572         // We could have an outer resolution that succeeded,
 573         // but with generic parameters that failed.
 574         // Recurse into the segments so we catch those too.
 575         intravisit::walk_path(self, path);
 576     }
 577 }
 578
 579 /// `DefId` or parameter index (`ty::ParamTy.index`) of a synthetic type parameter
 580 /// for `impl Trait` in argument position.
 581 #[derive(Clone, Copy, PartialEq, Eq, Hash, PartialOrd, Ord)]
 582 crate enum ImplTraitParam {
 583     DefId(DefId),
 584     ParamIndex(u32),
 585 }
 586
 587 impl From<DefId> for ImplTraitParam {
 588     fn from(did: DefId) -> Self {
 589         ImplTraitParam::DefId(did)
 590     }
 591 }
 592
 593 impl From<u32> for ImplTraitParam {
 594     fn from(idx: u32) -> Self {
 595         ImplTraitParam::ParamIndex(idx)
 596     }
 597 }