diff options
| author | Daniel Martí <mvdan@mvdan.cc> | 2019-09-19 15:31:03 +0100 |
|---|---|---|
| committer | Daniel Martí <mvdan@mvdan.cc> | 2019-09-23 16:36:10 +0000 |
| commit | 870080752d175d5cede350486acf36213c64f35c (patch) | |
| tree | 9fdf610cb118ed31f29a5c2f572870808cd18b49 /src/cmd/compile/internal/ssa/rewrite386splitload.go | |
| parent | 34fe8295c5ac099109d7f8a1f967edec96f2a71d (diff) | |
| download | go-870080752d175d5cede350486acf36213c64f35c.tar.gz go-870080752d175d5cede350486acf36213c64f35c.zip | |
cmd/compile: reduce rulegen's output by 200 KiB
First, renove unnecessary "// cond:" lines from the generated files.
This shaves off about ~7k lines.
Second, join "if cond { break }" statements via "||", which allows us to
deduplicate a large number of them. This shaves off another ~25k lines.
This change is not for readability or simplicity; but rather, to avoid
unnecessary verbosity that makes the generated files larger. All in all,
git reports that the generated files overall weigh ~200KiB less, or
about 2.7% less.
While at it, add a -trace flag to rulegen.
Updates #33644.
Change-Id: I3fac0290a6066070cc62400bf970a4ae0929470a
Reviewed-on: https://go-review.googlesource.com/c/go/+/196498
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
Diffstat (limited to 'src/cmd/compile/internal/ssa/rewrite386splitload.go')
| -rw-r--r-- | src/cmd/compile/internal/ssa/rewrite386splitload.go | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/cmd/compile/internal/ssa/rewrite386splitload.go b/src/cmd/compile/internal/ssa/rewrite386splitload.go index b6c1989d52..2698e35834 100644 --- a/src/cmd/compile/internal/ssa/rewrite386splitload.go +++ b/src/cmd/compile/internal/ssa/rewrite386splitload.go @@ -24,7 +24,6 @@ func rewriteValue386splitload_Op386CMPBconstload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPBconstload {sym} [vo] ptr mem) - // cond: // result: (CMPBconst (MOVBload {sym} [offOnly(vo)] ptr mem) [valOnly(vo)]) for { vo := v.AuxInt @@ -46,7 +45,6 @@ func rewriteValue386splitload_Op386CMPBload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPBload {sym} [off] ptr x mem) - // cond: // result: (CMPB (MOVBload {sym} [off] ptr mem) x) for { off := v.AuxInt @@ -69,7 +67,6 @@ func rewriteValue386splitload_Op386CMPLconstload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPLconstload {sym} [vo] ptr mem) - // cond: // result: (CMPLconst (MOVLload {sym} [offOnly(vo)] ptr mem) [valOnly(vo)]) for { vo := v.AuxInt @@ -91,7 +88,6 @@ func rewriteValue386splitload_Op386CMPLload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPLload {sym} [off] ptr x mem) - // cond: // result: (CMPL (MOVLload {sym} [off] ptr mem) x) for { off := v.AuxInt @@ -114,7 +110,6 @@ func rewriteValue386splitload_Op386CMPWconstload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPWconstload {sym} [vo] ptr mem) - // cond: // result: (CMPWconst (MOVWload {sym} [offOnly(vo)] ptr mem) [valOnly(vo)]) for { vo := v.AuxInt @@ -136,7 +131,6 @@ func rewriteValue386splitload_Op386CMPWload_0(v *Value) bool { b := v.Block typ := &b.Func.Config.Types // match: (CMPWload {sym} [off] ptr x mem) - // cond: // result: (CMPW (MOVWload {sym} [off] ptr mem) x) for { off := v.AuxInt |