[EXPERIMENT] inline fmt::write for Arguments without placeholders#152542
[EXPERIMENT] inline fmt::write for Arguments without placeholders#152542joboet wants to merge 1 commit intorust-lang:mainfrom
fmt::write for Arguments without placeholders#152542Conversation
|
@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.
[EXPERIMENT] inline `fmt::write` for templateless `Arguments`
fmt::write for templateless Argumentsfmt::write for Arguments without placeholders
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (13766b5): comparison URL. Overall result: ❌✅ regressions and improvements - please read the text belowBenchmarking 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. Next Steps: If you can justify the regressions found in this try perf run, please do so in sufficient writing along with @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 (primary -0.4%, secondary -3.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 4.9%)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: 480.915s -> 484.228s (0.69%) |
| } | ||
| } | ||
|
|
||
| if let Some(s) = fmt.as_str() { output.write_str(s) } else { complex(output, fmt) } |
There was a problem hiding this comment.
Did you mean to use as_statically_known_str?
No description provided.