Skip to content

cmd/compile: 1.15 heap allocations regression when calling Write on os.Stdout [1.15 backport] #41543

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.

Sign up for GitHub

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
Closed
gopherbot opened this issue Sep 21, 2020 · 3 comments
Closed

cmd/compile: 1.15 heap allocations regression when calling Write on os.Stdout [1.15 backport] #41543

gopherbot opened this issue Sep 21, 2020 · 3 comments
Labels
CherryPickApproved Used during the release process for point releases FrozenDueToAge
Milestone
Go1.15.3

Comments

@gopherbot
Copy link
Contributor

@ianlancetaylor requested issue #41474 to be considered for backport to the next 1.15 minor release.

@gopherbot Please open a backport issue for 1.15.

This is an unexpected and unavoidable performance regression in 1.15: using Write to write a []byte forces the []byte to escape into the heap. That did not happen in 1.14.
@gopherbot gopherbot added the CherryPickCandidate Used during the release process for point releases label Sep 21, 2020
@gopherbot gopherbot mentioned this issue Sep 21, 2020
Closed
@gopherbot gopherbot added this to the Go1.15.3 milestone Sep 21, 2020
@gopherbot
Copy link
Contributor Author

Change https://golang.org/cl/256418 mentions this issue: [release-branch.go1.15] internal/poll: adjust ignoringEINTR to avoid slice escape

@cagedmantis
Copy link
Contributor

Approved because this is a serious issue without a workaround.

@cagedmantis cagedmantis added the CherryPickApproved Used during the release process for point releases label Sep 24, 2020
@gopherbot gopherbot removed the CherryPickCandidate Used during the release process for point releases label Sep 24, 2020
@gopherbot
Copy link
Contributor Author

Closed by merging fa01c8a to release-branch.go1.15.

gopherbot pushed a commit that referenced this issue Sep 24, 2020
@ianlancetaylor
[release-branch.go1.15] internal/poll: adjust ignoringEINTR to avoid …
fa01c8a 
…slice escape

The 1.15 compiler is not quite smart enough to see that the byte slice
passed to ignoringEINTR does not escape. This ripples back up to user
code which would see a byte slice passed to os.(*File).Write escape,
which did not happen in 1.14.

Rather than backport some moderately complex compiler fixes, rewrite
the code slightly so that the 1.15 compiler is able to see that the
slice does not escape.

This is not a backport from tip, where the code is already different.
The test for this will be on tip, where we will most likely change the
compiler to understand this kind of code.

Fixes #41543
For #41474

Change-Id: I6c78164229fea7794e7edba512bfd7034a0b91c3
Reviewed-on: https://go-review.googlesource.com/c/go/+/256418
Trust: Ian Lance Taylor <[email protected]>
Reviewed-by: Matthew Dempsky <[email protected]>
@ploxiln ploxiln mentioned this issue Sep 28, 2020
Closed
claucece pushed a commit to claucece/go that referenced this issue Oct 22, 2020
@ianlancetaylor @claucece
[release-branch.go1.15] internal/poll: adjust ignoringEINTR to avoid …
c186848 
…slice escape

The 1.15 compiler is not quite smart enough to see that the byte slice
passed to ignoringEINTR does not escape. This ripples back up to user
code which would see a byte slice passed to os.(*File).Write escape,
which did not happen in 1.14.

Rather than backport some moderately complex compiler fixes, rewrite
the code slightly so that the 1.15 compiler is able to see that the
slice does not escape.

This is not a backport from tip, where the code is already different.
The test for this will be on tip, where we will most likely change the
compiler to understand this kind of code.

Fixes golang#41543
For golang#41474

Change-Id: I6c78164229fea7794e7edba512bfd7034a0b91c3
Reviewed-on: https://go-review.googlesource.com/c/go/+/256418
Trust: Ian Lance Taylor <[email protected]>
Reviewed-by: Matthew Dempsky <[email protected]>
@EricByers EricByers mentioned this issue Nov 3, 2020
Open
@golang golang locked and limited conversation to collaborators Sep 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
CherryPickApproved Used during the release process for point releases FrozenDueToAge
Projects
None yet
Milestone
Go1.15.3
Development

No branches or pull requests
2 participants
@cagedmantis @gopherbot