Clean up the eager formatting API#153508
Clean up the eager formatting API#153508JonathanBrouwer wants to merge 10 commits intorust-lang:mainfrom
Conversation
|
Some changes occurred to MIR optimizations cc @rust-lang/wg-mir-opt
cc @davidtwco, @TaKO8Ki Some changes occurred to the CTFE / Miri interpreter cc @rust-lang/miri Some changes occurred to constck cc @fee1-dead
cc @davidtwco, @TaKO8Ki Some changes occurred to the CTFE machinery |
This comment has been minimized.
This comment has been minimized.
|
@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.
Clean up the eager formatting API
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (48c772e): comparison URL. Overall result: no relevant changes - no action neededBenchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf. @bors rollup=never Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (secondary 4.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 481.299s -> 478.644s (-0.55%) |
|
@bors r=GuillaumeGomez rollup |
…mat, r=GuillaumeGomez Clean up the eager formatting API For rust-lang#151366 (comment) Previously eager formatting worked by throwing the arguments into a diag, formatting, then removing the args again. This is ugly so instead we now just do the formatting completely separately. This PR has nice commits, so I recommend reviewing commit by commit. r? @GuillaumeGomez
For #151366 (comment)
Previously eager formatting worked by throwing the arguments into a diag, formatting, then removing the args again. This is ugly so instead we now just do the formatting completely separately.
This PR has nice commits, so I recommend reviewing commit by commit.
r? @GuillaumeGomez