Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3656)

Issue 10305043: fmt: use sync.Cache

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by dvyukov
Modified:
10 years, 11 months ago
Reviewers:
Visibility:
Public.

Description

fmt: use sync.Cache benchmark old ns/op new ns/op delta BenchmarkSprintfEmpty 77 69 -10.22% BenchmarkSprintfEmpty-2 338 35 -89.44% BenchmarkSprintfEmpty-4 426 23 -94.48% BenchmarkSprintfEmpty-8 460 22 -95.22% BenchmarkSprintfEmpty-16 436 16 -96.28% BenchmarkSprintfEmpty-32 435 11 -97.47% BenchmarkSprintfEmpty-64 425 9 -97.74% BenchmarkSprintfString 261 232 -11.11% BenchmarkSprintfString-2 298 165 -44.63% BenchmarkSprintfString-4 555 85 -84.61% BenchmarkSprintfString-8 668 54 -91.78% BenchmarkSprintfString-16 673 39 -94.09% BenchmarkSprintfString-32 647 32 -94.98% BenchmarkSprintfString-64 691 30 -95.63%

Patch Set 1 #

Patch Set 2 : diff -r 2879112bff3d https://dvyukov%40google.com@code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -55 lines) Patch
M src/pkg/fmt/fmt_test.go View 1 2 chunks +36 lines, -15 lines 0 comments Download
M src/pkg/fmt/print.go View 1 2 chunks +3 lines, -37 lines 0 comments Download
M src/pkg/fmt/scan.go View 1 4 chunks +4 lines, -3 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b