incremental: migrate diagnostics by davidtwco · Pull Request #100754 · rust-lang/rust

davidtwco · 2022-08-19T14:10:43Z

Apply the diagnostic migration lints to more functions on Session, namely: span_warn, span_warn_with_code, warn note_without_error, span_note_without_error, struct_note_without_error.
Add impls of IntoDiagnosticArg for std::io::Error, std::path::Path and std::path::PathBuf.
Migrate the rustc_incremental crate's diagnostics to translatable diagnostic structs.

r? @compiler-errors
cc #100717

compiler-errors · 2022-08-19T22:34:42Z

r=me, rebased after that fatal pr lands

davidtwco · 2022-08-22T15:43:01Z

@bors r=compiler-errors

bors · 2022-08-22T15:43:03Z

📌 Commit 3becaaf has been approved by compiler-errors

It is now in the queue for this repository.

…s, r=davidtwco save_analysis: Migrate diagnostic * Migrate the `rustc_save_analysis` crate's diagnostic to translatable diagnostic structs. Depends on rust-lang#100694 and rust-lang#100754 for #[fatal(..)] support, then rust-lang@aa68eb4, rust-lang@f5219a3, rust-lang@7da52f6 can be removed. (I copied commits from rust-lang#100754)

… r=compiler-errors incremental: migrate diagnostics - Apply the diagnostic migration lints to more functions on `Session`, namely: `span_warn`, `span_warn_with_code`, `warn` `note_without_error`, `span_note_without_error`, `struct_note_without_error`. - Add impls of `IntoDiagnosticArg` for `std::io::Error`, `std::path::Path` and `std::path::PathBuf`. - Migrate the `rustc_incremental` crate's diagnostics to translatable diagnostic structs. r? ```@compiler-errors``` cc rust-lang#100717

compiler-errors · 2022-08-27T03:45:31Z

@bors r- failed in rollup I think

davidtwco · 2023-01-27T10:58:29Z

There haven't been any significant changes to this since last approved, but there was a bors failure that I couldn't reproduce. It's been a while so maybe that's gone away?

@bors r=compiler-errors

bors · 2023-01-27T10:58:31Z

📌 Commit af8ec72 has been approved by compiler-errors

It is now in the queue for this repository.

matthiaskrgr · 2023-01-28T10:09:43Z

@bors rollup=iffy might fail ci

bors · 2023-01-28T15:54:10Z

⌛ Testing commit af8ec72 with merge 1b307e9d23836e2a24c87220eaf52ecb0615f3b1...

bors · 2023-01-28T16:07:52Z

💔 Test failed - checks-actions

Support for emission of notes was added in f8ebc72 but `emit_note` and `create_note` functions weren't added to `Handler`. Signed-off-by: David Wood <david.wood@huawei.com>

Implement `IntoDiagnosticArg` for `&'a T` when `T` implements `IntoDiagnosticArg` and `Clone`. Makes it easier to write diagnostic structs that borrow something which implements `IntoDiagnosticArg`. Signed-off-by: David Wood <david.wood@huawei.com>

Forward the `Display` implementation for `CrateType` to `IntoDiagnosticArg` so that it can be used in diagnostic structs. Signed-off-by: David Wood <david.wood@huawei.com>

Apply the diagnostic migration lint to more functions on `Session`. Signed-off-by: David Wood <david.wood@huawei.com>

Migrate the `rustc_incremental` crate's diagnostics to translatable diagnostic structs. Signed-off-by: David Wood <david.wood@huawei.com>

davidtwco · 2023-01-30T17:12:34Z

Okay, managed to reproduce this and fix it - diagnostic args weren't being hashed as part of Diagnostic, so we were deduplicating a warning for unsupported crate types, which Cargo parses.

davidtwco · 2023-01-31T09:40:31Z

@bors r=compiler-errors

bors · 2023-01-31T09:40:33Z

📌 Commit 2ff4664 has been approved by compiler-errors

It is now in the queue for this repository.

bors · 2023-01-31T09:57:51Z

⌛ Testing commit 2ff4664 with merge 79595d1652139e0cdb8fb3ff3ea67f2c9a190a0a...

bors · 2023-01-31T10:08:45Z

💔 Test failed - checks-actions

davidtwco · 2023-01-31T10:11:37Z

I think that's spurious - nothing in this PR should cause a failure in building LLVM?

@bors retry

bors · 2023-01-31T10:21:02Z

⌛ Testing commit 2ff4664 with merge a64ef7d...

rust-log-analyzer · 2023-01-31T12:54:39Z

The job x86_64-mingw-2 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[2620/3025] Building CXX object tools/dsymutil/CMakeFiles/dsymutil.dir/MachOUtils.cpp.obj
[2621/3025] Building CXX object tools/dsymutil/CMakeFiles/dsymutil.dir/SymbolMap.cpp.obj
[2622/3025] Linking CXX static library lib\libLLVMGlobalISel.a
[2623/3025] Linking CXX static library lib\libLLVMAArch64CodeGen.a
FAILED: lib/libLLVMAArch64CodeGen.a 
cmd.exe /C "cd . && "C:\Program Files\CMake\bin\cmake.exe" -E rm -f lib\libLLVMAArch64CodeGen.a && D:\a\rust\rust\mingw64\bin\ar.exe qc lib\libLLVMAArch64CodeGen.a  lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64CallLowering.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64GlobalISelUtils.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64InstructionSelector.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64LegalizerInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64O0PreLegalizerCombiner.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64PreLegalizerCombiner.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64PostLegalizerCombiner.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64PostLegalizerLowering.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64PostSelectOptimize.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/GISel/AArch64RegisterBankInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64A57FPLoadBalancing.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64AdvSIMDScalarPass.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64AsmPrinter.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64BranchTargets.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64CallingConvention.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64CleanupLocalDynamicTLSPass.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64CollectLOH.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64CondBrTuning.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ConditionalCompares.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64DeadRegisterDefinitionsPass.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ExpandImm.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ExpandPseudoInsts.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64FalkorHWPFFix.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64FastISel.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64A53Fix835769.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64FrameLowering.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64CompressJumpTables.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ConditionOptimizer.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64RedundantCopyElimination.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ISelDAGToDAG.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64ISelLowering.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64InstrInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64LoadStoreOptimizer.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64LowerHomogeneousPrologEpilog.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64MachineFunctionInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64MachineScheduler.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64MacroFusion.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64MIPeepholeOpt.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64MCInstLower.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64PromoteConstant.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64PBQPRegAlloc.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64RegisterInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64SLSHardening.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64SelectionDAGInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64SpeculationHardening.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64StackTagging.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64StackTaggingPreRA.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64StorePairSuppress.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64Subtarget.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64TargetMachine.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64TargetObjectFile.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64TargetTransformInfo.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/SVEIntrinsicOpts.cpp.obj lib/Target/AArch64/CMakeFiles/LLVMAArch64CodeGen.dir/AArch64SIMDInstrOpt.cpp.obj && D:\a\rust\rust\mingw64\bin\ranlib.exe lib\libLLVMAArch64CodeGen.a && cd ."
D:\a\rust\rust\mingw64\bin\ar.exe: could not create temporary file whilst writing archive: no more archived files
[2624/3025] Linking CXX static library lib\libLLVMBPFCodeGen.a
[2625/3025] Linking CXX static library lib\libLLVMMSP430CodeGen.a
[2626/3025] Building CXX object tools/lli/CMakeFiles/lli.dir/ExecutionUtils.cpp.obj
[2627/3025] Linking CXX static library lib\libLLVMHexagonCodeGen.a
[2627/3025] Linking CXX static library lib\libLLVMHexagonCodeGen.a
[2628/3025] Building CXX object tools/lli/CMakeFiles/lli.dir/lli.cpp.obj
[2629/3025] Linking CXX executable bin\llvm-profdata.exe
[2630/3025] Linking CXX static library lib\libLLVMPasses.a
ninja: build stopped: subcommand failed.
command did not execute successfully, got: exit code: 1


build script failed, must exit now', C:\Users\runneradmin\.cargo\registry\src\github.com-1ecc6299db9ec823\cmake-0.1.48\src\lib.rs:975:5
 finished in 237.369 seconds
Build completed unsuccessfully in 0:05:17
Build completed unsuccessfully in 0:05:17
make: *** [Makefile:80: ci-mingw-subset-2] Error 1

bors · 2023-01-31T13:21:59Z

☀️ Test successful - checks-actions
Approved by: compiler-errors
Pushing a64ef7d to master...

rust-timer · 2023-01-31T14:41:04Z

Finished benchmarking commit (a64ef7d): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	2.3%	[2.3%, 2.3%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-2.0%	[-2.4%, -1.3%]	3
Improvements ✅ (secondary)	-2.2%	[-5.1%, -0.5%]	19
All ❌✅ (primary)	-0.9%	[-2.4%, 2.3%]	4

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-2.1%	[-2.1%, -2.1%]	1
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-2.1%	[-2.1%, -2.1%]	1

rust-highfive assigned compiler-errors Aug 19, 2022

rustbot added A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 19, 2022

rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 19, 2022

wonchulee mentioned this pull request Aug 19, 2022

save_analysis: Migrate diagnostic #100780

Merged

compiler-errors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 21, 2022

This comment was marked as resolved.

Sign in to view

davidtwco mentioned this pull request Aug 22, 2022

Migrate rustc_interface diagnostics #100808

Merged

davidtwco force-pushed the translation-incremental branch 2 times, most recently from 20c0940 to 3becaaf Compare August 22, 2022 14:55

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 22, 2022

This was referenced Aug 23, 2022

Diagnostic Translation #100717

Open

Migrate rustc_driver to SessionDiagnostic #100890

Merged

Dylan-DPC mentioned this pull request Aug 25, 2022

Rollup of 7 pull requests #100985

Closed

davidtwco mentioned this pull request Aug 25, 2022

Migrate rustc_metadata to SessionDiagnostics #100928

Merged

compiler-errors mentioned this pull request Aug 27, 2022

Rollup of 12 pull requests #101073

Closed

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 27, 2022

This comment was marked as resolved.

Sign in to view

This comment has been minimized.

Sign in to view

This comment was marked as resolved.

Sign in to view

This was referenced Jan 27, 2023

Rollup of 9 pull requests #107390

Closed

Rollup of 6 pull requests #107399

Closed

This comment has been minimized.

Sign in to view

davidtwco added 5 commits January 30, 2023 17:11

errors: add emit_note/create_note on Handler

59cc5e5

Support for emission of notes was added in f8ebc72 but `emit_note` and `create_note` functions weren't added to `Handler`. Signed-off-by: David Wood <david.wood@huawei.com>

session: impl IntoDiagnosticArg for CrateType

d450048

Forward the `Display` implementation for `CrateType` to `IntoDiagnosticArg` so that it can be used in diagnostic structs. Signed-off-by: David Wood <david.wood@huawei.com>

session: diagnostic migration lint on more fns

2575b1a

Apply the diagnostic migration lint to more functions on `Session`. Signed-off-by: David Wood <david.wood@huawei.com>

incremental: migrate diagnostics

2ff4664

Migrate the `rustc_incremental` crate's diagnostics to translatable diagnostic structs. Signed-off-by: David Wood <david.wood@huawei.com>

This was referenced Jan 31, 2023

[WIP] make nightly compilers able to parallelize #101566

Closed

Partial stabilization of once_cell #105587

Merged

rustc_parse: diagnostics migration, v4 #105670

Merged

Monomorphize in check mode to report (almost) all PMEs #107510

Closed

erikdesjardins mentioned this pull request Feb 19, 2023

CI: spurious error building LLVM on mingw: ranlib.exe: could not create temporary file whilst writing archive: no more archived files #108227

Open

Uh oh!

Conversation

davidtwco commented Aug 19, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

compiler-errors commented Aug 19, 2022

Uh oh!

This comment was marked as resolved.

davidtwco commented Aug 22, 2022

Uh oh!

bors commented Aug 22, 2022

Uh oh!

compiler-errors commented Aug 27, 2022

Uh oh!

This comment was marked as resolved.

This comment has been minimized.

This comment was marked as resolved.

davidtwco commented Jan 27, 2023

Uh oh!

bors commented Jan 27, 2023

Uh oh!

matthiaskrgr commented Jan 28, 2023

Uh oh!

bors commented Jan 28, 2023

Uh oh!

bors commented Jan 28, 2023

Uh oh!

This comment has been minimized.

davidtwco commented Jan 30, 2023

Uh oh!

davidtwco commented Jan 31, 2023

Uh oh!

bors commented Jan 31, 2023

Uh oh!

bors commented Jan 31, 2023

Uh oh!

bors commented Jan 31, 2023

Uh oh!

davidtwco commented Jan 31, 2023

Uh oh!

bors commented Jan 31, 2023

Uh oh!

rust-log-analyzer commented Jan 31, 2023

Uh oh!

bors commented Jan 31, 2023

Uh oh!

rust-timer commented Jan 31, 2023

Overall result: no relevant changes - no action needed

Instruction count

Max RSS (memory usage)

Cycles

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

davidtwco commented Aug 19, 2022 •

edited

Loading