• David Crawshaw's avatar
    encoding/json: copy-on-write cacheTypeFields · 5f209aba
    David Crawshaw authored
    Swtich from a sync.RWMutex to atomic.Value for cacheTypeFields.
    
    On GOARCH=386, this recovers most of the remaining performance
    difference from the 1.6 release. Compared with tip on linux/386:
    
    	name            old time/op    new time/op    delta
    	CodeDecoder-40    92.8ms ± 1%    87.7ms ± 1%  -5.50%  (p=0.000 n=10+10)
    
    	name            old speed      new speed      delta
    	CodeDecoder-40  20.9MB/s ± 1%  22.1MB/s ± 1%  +5.83%  (p=0.000 n=10+10)
    
    With more time and care, I believe more of the JSON decoder's work
    could be shifted so it is done before decoding, and independent of
    the number of bytes processed. Maybe someone could explore that for
    Go 1.8.
    
    For #16117.
    
    Change-Id: I049655b2e5b76384a0d5f4b90e3ec7cc8d8c4340
    Reviewed-on: https://go-review.googlesource.com/24472
    Run-TryBot: Dmitry Vyukov <dvyukov@google.com>
    Reviewed-by: 's avatarDmitry Vyukov <dvyukov@google.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    5f209aba
Name
Last commit
Last update
..
testdata Loading commit data...
bench_test.go Loading commit data...
decode.go Loading commit data...
decode_test.go Loading commit data...
encode.go Loading commit data...
encode_test.go Loading commit data...
example_test.go Loading commit data...
fold.go Loading commit data...
fold_test.go Loading commit data...
indent.go Loading commit data...
number_test.go Loading commit data...
scanner.go Loading commit data...
scanner_test.go Loading commit data...
stream.go Loading commit data...
stream_test.go Loading commit data...
tagkey_test.go Loading commit data...
tags.go Loading commit data...
tags_test.go Loading commit data...