Merge pull request #1134 from Mark-Simulacrum/triage

Triage for this week

Author: Mark-Simulacrum

triage/2021-12-28.md

@@ -0,0 +1,53 @@
+# 2021-12-28 Triage Log
+
+Relatively quiet week, mostly rustdoc improvements.
+
+Triage done by **@simulacrum**.
+Revision range: [3d57c61a9e04dcd3df633f41142009d6dcad4399..e91ad5fc62bdee4a29c18baa5fad2ca42fc91bf4](https://perf.rust-lang.org/?start=3d57c61a9e04dcd3df633f41142009d6dcad4399&end=e91ad5fc62bdee4a29c18baa5fad2ca42fc91bf4&absolute=false&stat=instructions%3Au)
+
+2 Regressions, 1 Improvements, 6 Mixed; 0 of them in rollups
+26 comparisons made in total
+
+#### Regressions
+
+Fix bad caching of `~const Drop` bounds [#92149](https://github.com/rust-lang/rust/issues/92149)
+- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=3d57c61a9e04dcd3df633f41142009d6dcad4399&end=8ad3c1dd1d47f9ce7dfdf4a14c70c67e1790b0f5&stat=instructions:u) (up to 2.6% on `incr-patched: println` builds of `regression-31157`)
+- Necessary fix for correctness.
+
+Add `#[inline]` modifier to `TypeId::of` [#92135](https://github.com/rust-lang/rust/issues/92135)
+- Large regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=aad4f1039fd5d6bf961ed08eebc6eb69b577f1be&end=475b00aa4037461b506539a06d15ca6091b461a7&stat=instructions:u) (up to 4.4% on `full` builds of `keccak`)
+- Reverted in a later pull request, unclear what the exact cause of the
+  regression is.
+
+#### Improvements
+
+:arrow_up: rust-analyzer [#92247](https://github.com/rust-lang/rust/issues/92247)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=83bde52116c704fa2a5244e035c6f61a1fea8a85&end=67491a22dbafc275b1549c31876c1fd356bd51cc&stat=instructions:u) (up to -1.2% on `incr-unchanged` builds of `deeply-nested-async`)
+- Likely spurious change, rust-analyzer performance is not directly related.
+
+Remove `PartialOrd`, `Ord` from `LocalDefId` [#90408](https://github.com/rust-lang/rust/issues/90408)
+- Large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=34926f0a1681458588a2d4240c0715ef9eff7d35&end=e98309298d927307c5184f4869604bd068d26183&stat=instructions:u) (up to -3.7% on `full` builds of `deeply-nested-async`)
+- Overall an improvement; largely a correctness change as relative ordering of
+  DefId's isn't tracked by incremental compilation.
+
+Do not display `~const Drop` in rustdoc [#92229](https://github.com/rust-lang/rust/issues/92229)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=475b00aa4037461b506539a06d15ca6091b461a7&end=83bde52116c704fa2a5244e035c6f61a1fea8a85&stat=instructions:u) (up to -0.8% on `incr-unchanged` builds of `externs`)
+- Likely to be just noise, no clear explanation for changes to rustc performance
+  from rustdoc changes.
+
+Rustdoc: use `is_doc_hidden` method on more places [#92227](https://github.com/rust-lang/rust/issues/92227)
+- Very large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=67491a22dbafc275b1549c31876c1fd356bd51cc&end=c096176fb411c90a8b0226901c11e7edd131192f&stat=instructions:u) (up to -9.3% on `full` builds of `helloworld`)
+
+intra-doc: Use an enum to represent URL fragments [#92088](https://github.com/rust-lang/rust/issues/92088)
+- Small improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=41ce641a407965ca0709f9f2d60faae3a56755c2&end=cc65bf3ded93352fd4693a9c58b84a60721637a6&stat=instructions:u) (up to -0.8% on `incr-unchanged` builds of `externs`)
+
+rustc_metadata: Switch crate data iteration from a callback to iterator [#92159](https://github.com/rust-lang/rust/issues/92159)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=cc65bf3ded93352fd4693a9c58b84a60721637a6&end=e91ad5fc62bdee4a29c18baa5fad2ca42fc91bf4&stat=instructions:u) (up to -1.3% on `incr-unchanged` builds of `deeply-nested-async`)
+
+#### Mixed
+
+Backport LLVM changes to disable deferred inlining [#92110](https://github.com/rust-lang/rust/issues/92110)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=489296d82561f596c278e90edc10eb56168ab623&end=77497c74f9268ccf91d7b4c17f23bf07117d7433&stat=instructions:u) (up to -8.3% on `incr-patched: println` builds of `regression-31157`)
+- Large regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=489296d82561f596c278e90edc10eb56168ab623&end=77497c74f9268ccf91d7b4c17f23bf07117d7433&stat=instructions:u) (up to 2.8% on `full` builds of `deeply-nested`)
+- Avoids an extreme regression in LLVM, which was blocking migration onto the
+  new pass manager (and so newer rustc versions) for some crates.