Skip to content
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

Jump to bottom

cmd/compile: inline multiplication corrupts an argument on arm [1.10 backport] #28959

Closed
andybons opened this issue Nov 26, 2018 · 3 comments
Closed

cmd/compile: inline multiplication corrupts an argument on arm [1.10 backport] #28959

andybons opened this issue Nov 26, 2018 · 3 comments
Labels
FrozenDueToAge
Milestone
Go1.10.7

Comments

@andybons
Copy link
Member

Manually opening this due to gopherbot not tracking follow-up backport requests.

Issue #28688 is to be considered for backport to the next 1.10 minor release.

@josharian @milanknezevic
@andybons andybons added the CherryPickApproved Used during the release process for point releases label Nov 26, 2018
@andybons andybons added this to the Go1.10.6 milestone Nov 26, 2018
@gopherbot
Copy link

Change https://golang.org/cl/151343 mentions this issue: [release-branch.go1.10] cmd/compile/internal/gc: OMUL should be evaluated when using soft-float

@dmitshur dmitshur modified the milestones: Go1.10.6, Go1.10.7 Dec 13, 2018
gopherbot pushed a commit that referenced this issue Dec 14, 2018
@FiloSottile
[release-branch.go1.10] cmd/compile/internal/gc: OMUL should be evalu…
d74e69c 
…ated when using soft-float

When using soft-float, OMUL might be rewritten to function call
so we should ensure it was evaluated first.

Updates #28688
Fixes #28959

Change-Id: I30b87501782fff62d35151f394a1c22b0d490c6c
Reviewed-on: https://go-review.googlesource.com/c/148837
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
(cherry picked from commit c92e73b)
Reviewed-on: https://go-review.googlesource.com/c/151343
Reviewed-by: Filippo Valsorda <filippo@golang.org>
@gopherbot
Copy link

Closed by merging d74e69c to release-branch.go1.10.

@gopherbot
Copy link

Change https://golang.org/cl/154297 mentions this issue: Revert "[release-branch.go1.10] cmd/compile/internal/gc: OMUL should be evaluated when using soft-float"

gopherbot pushed a commit that referenced this issue Dec 14, 2018
@FiloSottile
Revert "[release-branch.go1.10] cmd/compile/internal/gc: OMUL should …
a9ccbe0 
…be evaluated when using soft-float"

This reverts commit d74e69c.

Reason for revert: this fix has been backported to 1.11, and that makes
it ineligible for backport to 1.10, since upgrading to 1.11 is a valid
workaround according to https://github.com/golang/go/wiki/MinorReleases.

Also, this triggers a bug on Wasm that would require a backport of
CL 149965, too.

Updates #28688
Updates #28959

Change-Id: I4e56554ea27b5db83bdb1e9d2103dd24e3da8cfc
Reviewed-on: https://go-review.googlesource.com/c/154297
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Run-TryBot: Cherry Zhang <cherryyz@google.com>
@FiloSottile FiloSottile removed the CherryPickApproved Used during the release process for point releases label Dec 14, 2018
@golang golang locked and limited conversation to collaborators Dec 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.10.7
Development

No branches or pull requests
4 participants
@andybons @FiloSottile @dmitshur @gopherbot