various small rustc_expand cleanups#155311
Conversation
|
rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer |
|
r? @BoxyUwU rustbot has assigned @BoxyUwU. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
r? me just wondering about how much this affects perf @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.
various small `rustc_expand` cleanups
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (dc646e9): 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 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 1.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 4.9%, secondary 0.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 0.1%, secondary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 489.296s -> 491.084s (0.37%) |
|
@bors r=Kivooeo,petrochenkov rollup |
…uwer Rollup of 5 pull requests Successful merges: - #155331 (Reformat `top_level_options!` and `options!` macro declarations) - #155332 (bump std libc to `0.2.185`) - #155286 (attribute cleanup: rustc_confusables) - #155306 (`CValue::zst()` - add missing "ZST" in docs) - #155311 (various small `rustc_expand` cleanups)
…uwer Rollup of 5 pull requests Successful merges: - #155331 (Reformat `top_level_options!` and `options!` macro declarations) - #155332 (bump std libc to `0.2.185`) - #155286 (attribute cleanup: rustc_confusables) - #155306 (`CValue::zst()` - add missing "ZST" in docs) - #155311 (various small `rustc_expand` cleanups)
Rollup merge of #155311 - cyrgani:expand-clean, r=Kivooeo,petrochenkov various small `rustc_expand` cleanups Each commit should be reviewable on its own.
Each commit should be reviewable on its own.