New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/compile: combine phielim and copyelim into a single pass #63818
Conversation
This PR (HEAD: 45c9cd2) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 1: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
45c9cd2
to
8db565a
Compare
This PR (HEAD: 8db565a) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
Message from 汪少军: Patch Set 2: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 2: Auto-Submit+1 Code-Review+2 Commit-Queue+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 2: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2023-10-31T16:20:19Z","revision":"0b6c626255aa69bbc6e9d738c1f395243a22fd95"} Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 2: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 2: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 2: This CL has failed the run. Reason: Tryjob golang/try/x_tools-gotip-linux-amd64 has failed with summary (view all results): FAILURE
error: failed to run "test golang.org/x/tools module": exit status 1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 2: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Yi Yang: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Yi Yang: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Yi Yang: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
3e7c537
to
6acd078
Compare
This PR (HEAD: 6acd078) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
Message from Keith Randall: Patch Set 3: Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 3: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2023-11-03T18:12:13Z","revision":"d3da99223ed47e84dcf5360d3a94c552bd96b142"} Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 3: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 3: This CL has failed the run. Reason: Tryjob golang/try/x_tools-gotip-linux-amd64 has failed with summary (view all results): FAILURE
error: failed to run "test golang.org/x/tools module": exit status 1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 3: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
6acd078
to
306e176
Compare
This PR (HEAD: 306e176) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
Message from 汪少军: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 4: Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 4: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2023-11-09T05:08:04Z","revision":"191dcfe62cc80932d45196148d782fcd6f07927e"} Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 4: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 4: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 4: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from 汪少军: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Keith Randall: Patch Set 5: Code-Review+2 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Heschi Kreinick: Patch Set 5: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Carlos Amedee: Patch Set 5: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from jeffery wsj: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
306e176
to
7de91b4
Compare
This PR (HEAD: 7de91b4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
7de91b4
to
59f7911
Compare
This PR (HEAD: 59f7911) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
59f7911
to
5413f47
Compare
This PR (HEAD: 5413f47) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/538535. Important tips:
|
Message from Keith Randall: Patch Set 8: Auto-Submit+1 Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Message from Go LUCI: Patch Set 8: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-04-22T14:35:41Z","revision":"c16135f9143c09711c9af50a0fda09bbafad89e5"} Please don’t reply on this GitHub thread. Visit golang.org/cl/538535. |
Change-Id: Id21145b14169d28bac2144a31f6d3d9729f4be1e GitHub-Last-Rev: 5413f47 GitHub-Pull-Request: #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>
This PR is being closed because golang.org/cl/538535 has been merged. |
No description provided.