]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #100730 - CleanCut:diagnostics-rustc_monomorphize, r=davidtwco
authorRalf Jung <post@ralfj.de>
Wed, 31 Aug 2022 12:29:51 +0000 (14:29 +0200)
committerGitHub <noreply@github.com>
Wed, 31 Aug 2022 12:29:51 +0000 (14:29 +0200)
commit6c4bda6de456460520aa450d7ed10a1b3d9fa205
tree0e0cba4b4179f0f1b600a34412a28005ca1a7eb3
parent775e96970fa5c4af239becc16c78c8a37e8b25e9
parent845d56759c71391a23c73a328bea71b9342d7933
Rollup merge of #100730 - CleanCut:diagnostics-rustc_monomorphize, r=davidtwco

Migrate rustc_monomorphize to use SessionDiagnostic

### Description

- Migrates diagnostics in `rustc_monomorphize` to use `SessionDiagnostic`
- Adds an `impl IntoDiagnosticArg for PathBuf`

### TODO / Help!
- [x] I'm having trouble figuring out how to apply an optional note. ðŸ˜•  Help!?
  - Resolved. It was bad docs. Fixed in https://github.com/rust-lang/rustc-dev-guide/pull/1437/files
- [x] `errors:RecursionLimit` should be `#[fatal ...]`, but that doesn't exist so it's `#[error ...]` at the moment.
  - Maybe I can switch after this is merged in? --> https://github.com/rust-lang/rust/pull/100694
  - Or maybe I need to manually implement `SessionDiagnostic` instead of deriving it?
- [x] How does one go about converting an error inside of [a call to struct_span_lint_hir](https://github.com/rust-lang/rust/blob/8064a495086c2e63c0ef77e8e82fe3b9b5dc535f/compiler/rustc_monomorphize/src/collector.rs#L917-L927)?
- [x] ~What placeholder do you use in the fluent template to refer to the value in a vector? It seems like [this code](https://github.com/rust-lang/rust/blob/0b79f758c9aa6646606662a6d623a0752286cd17/compiler/rustc_macros/src/diagnostics/diagnostic_builder.rs#L83-L114) ought to have the answer (or something near it)...but I can't figure it out.~ You can't. Punted.
Cargo.lock
compiler/rustc_codegen_cranelift/src/base.rs
compiler/rustc_error_messages/src/lib.rs
compiler/rustc_monomorphize/src/collector.rs
compiler/rustc_monomorphize/src/partitioning/mod.rs