diff options
author | jeffery <superajun@gmail.com> | 2024-04-22 08:02:22 +0000 |
---|---|---|
committer | Gopher Robot <gobot@golang.org> | 2024-04-22 14:55:18 +0000 |
commit | 69aa1974f3c525b024ac1b13324d809909622246 (patch) | |
tree | 5a1be760654a14c99615c0eb4c80f7efd399ebde /src/cmd/compile/internal/ssa/compile.go | |
parent | 4bb67bc21eea06afadceec239bae6e5e40a9e759 (diff) | |
download | go-69aa1974f3c525b024ac1b13324d809909622246.tar.gz go-69aa1974f3c525b024ac1b13324d809909622246.zip |
cmd/compile: combine phielim and copyelim into a single pass
Change-Id: Id21145b14169d28bac2144a31f6d3d9729f4be1e
GitHub-Last-Rev: 5413f4753e5acb60db6a93cb3409047bddc8df6d
GitHub-Pull-Request: golang/go#63818
Reviewed-on: https://go-review.googlesource.com/c/go/+/538535
Auto-Submit: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Diffstat (limited to 'src/cmd/compile/internal/ssa/compile.go')
-rw-r--r-- | src/cmd/compile/internal/ssa/compile.go | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/cmd/compile/internal/ssa/compile.go b/src/cmd/compile/internal/ssa/compile.go index d125891f88..80ef53d085 100644 --- a/src/cmd/compile/internal/ssa/compile.go +++ b/src/cmd/compile/internal/ssa/compile.go @@ -455,10 +455,8 @@ commas. For example: // list of passes for the compiler var passes = [...]pass{ - // TODO: combine phielim and copyelim into a single pass? {name: "number lines", fn: numberLines, required: true}, - {name: "early phielim", fn: phielim}, - {name: "early copyelim", fn: copyelim}, + {name: "early phielim and copyelim", fn: copyelim}, {name: "early deadcode", fn: deadcode}, // remove generated dead code to avoid doing pointless work during opt {name: "short circuit", fn: shortcircuit}, {name: "decompose user", fn: decomposeUser, required: true}, @@ -496,8 +494,7 @@ var passes = [...]pass{ {name: "tighten tuple selectors", fn: tightenTupleSelectors, required: true}, {name: "lowered deadcode", fn: deadcode, required: true}, {name: "checkLower", fn: checkLower, required: true}, - {name: "late phielim", fn: phielim}, - {name: "late copyelim", fn: copyelim}, + {name: "late phielim and copyelim", fn: copyelim}, {name: "tighten", fn: tighten, required: true}, // move values closer to their uses {name: "late deadcode", fn: deadcode}, {name: "critical", fn: critical, required: true}, // remove critical edges |