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
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.