deeper rustc_builtin_macros cleanups - #161867
Conversation
|
Changes to the code generated for builtin derived traits. cc @nnethercote |
|
r? @adwinwhite rustbot has assigned @adwinwhite. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
deeper `rustc_builtin_macros` cleanups
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (cc25c3c): comparison URL. Overall result: ✅ improvements - no action neededBenchmarking means the PR may be perf-sensitive. It's automatically marked not fit for rolling up. Overriding is possible but disadvised: it risks changing compiler perf. @bors rollup=never rustc-perf Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary -1.0%, secondary -3.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -1.1%, secondary 0.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 476.343s -> 474.625s (-0.36%) |
979ef46 to
40d4c72
Compare
This comment has been minimized.
This comment has been minimized.
40d4c72 to
4178dac
Compare
This comment has been minimized.
This comment has been minimized.
4178dac to
3c3c3cd
Compare
As a followup to #161464, this contains a few more surface cleanups and begins simplifying the mess that
SubstructureFieldscurrently is by removing theStaticFieldstype in favor of just using theVariantDatadirectly.