[tools] go/ssa: remove redundant emitConv call

0 views
Skip to first unread message

Gopher Robot (Gerrit)

unread,
Jun 5, 2026, 5:06:01 PM (2 days ago) Jun 5
to Mark Freeman, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, golang...@luci-project-accounts.iam.gserviceaccount.com, Alan Donovan, golang-co...@googlegroups.com

Gopher Robot submitted the change

Unreviewed changes

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:
go/ssa: remove redundant emitConv call

The thunk is clearly already of type thunk.Signature, so the
conversion will always be a no-op. Thanks to Dominik Honnef
for reporting.
Change-Id: I58fdef051419862c3d419ecebb805bb239cd1d37
Reviewed-by: Alan Donovan <adon...@google.com>
Auto-Submit: Mark Freeman <markf...@google.com>
Files:
  • M go/ssa/builder.go
Change size: XS
Delta: 1 file changed, 1 insertion(+), 1 deletion(-)
Branch: refs/heads/master
Submit Requirements:
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: tools
Gerrit-Branch: master
Gerrit-Change-Id: I58fdef051419862c3d419ecebb805bb239cd1d37
Gerrit-Change-Number: 785700
Gerrit-PatchSet: 10
Gerrit-Owner: Mark Freeman <markf...@google.com>
Gerrit-Reviewer: Alan Donovan <adon...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Mark Freeman <markf...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages