Repository navigation
Conversation
`addr_of` sounds too much like the `addr_of!` macro or any other operation using raw pointers. It's also consistent with `ty_ref` and `expr_array_ref`.
|
@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.
`rustc_builtin_macros` cleanup, part 9
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (c82bfbd): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Benchmarking means the PR may be perf-sensitive. Consider adding rollup=never if this change is not fit for rolling up. Next, please: If you can, justify the regressions found in this try perf run in writing along with @bors rollup=iffy 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 (secondary -2.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -2.7%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 484.843s -> 484.517s (-0.07%) |
This matches the order in which it is actually written in code.
This allows removing some clones and simplifies the code since it only needs to work with up to two selflike arguments, which is all we need.
This removes several clones and works since each method def only gets expanded once. It also requires inlining `create_derived_impl` to avoid borrow checking errors.
c166c2b to
ccaad2a
Compare
This PR does two things: first, a lot of outdated, confusing and unhelpful things are renamed. The last few commits then inline most of the larger functions in
generic.rsintoexpand_ext, which allows the compiler to understand that several fields ofTraitDefandMethodDefare only used once and don't need to be cloned, which gives some small perf gains.Previous PR: #163485
r? @nnethercote