]> Cypherpunks repositories - gostls13.git/commit
cmd/compiler: skip some go/printer work in rulegen
authorDaniel Martí <mvdan@mvdan.cc>
Fri, 13 Sep 2019 14:45:18 +0000 (15:45 +0100)
committerDaniel Martí <mvdan@mvdan.cc>
Mon, 16 Sep 2019 21:37:16 +0000 (21:37 +0000)
commit0e0bff840e3cd041aa9d103c6135862faae9c03f
tree399e19b01ffa86acd7307a61d561872e23b074ca
parent357e8f83f868bd028df7f2f1c373b4d0c57d1408
cmd/compiler: skip some go/printer work in rulegen

We use go/format on the final output, so don't bother with the added
tabwriter work to align comments when using go/printer.

name     old time/op         new time/op         delta
Rulegen          2.53s ± 2%          2.48s ± 1%  -2.20%  (p=0.032 n=5+5)

name     old user-time/op    new user-time/op    delta
Rulegen          11.2s ± 1%          10.8s ± 0%  -3.72%  (p=0.008 n=5+5)

name     old sys-time/op     new sys-time/op     delta
Rulegen          218ms ±17%          207ms ±19%    ~     (p=0.548 n=5+5)

name     old peak-RSS-bytes  new peak-RSS-bytes  delta
Rulegen          184MB ± 3%          175MB ± 4%    ~     (p=0.056 n=5+5)

Change-Id: I53bad2ab15cace67415f2171fffcd13ed596e62b
Reviewed-on: https://go-review.googlesource.com/c/go/+/195219
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/cmd/compile/internal/ssa/gen/rulegen.go