proc: add regression test for issue #3548 #3553
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
derekparker
reviewed
Nov 3, 2023
|
|
||
| func TestCapturedVariable(t *testing.T) { | ||
| if !goversion.VersionAfterOrEqual(runtime.Version(), 1, 22) { | ||
| t.Skip("broken") |
There was a problem hiding this comment.
Can you add more context here, like broken in versions prior to 1.22 due to a compiler bug. Also, does this affect all versions < 1.22 or was the bug introduced in 1.21 and we didn't catch it? If the latter, we should be more specific in the skip.
Finally, can you add a link to the Github issue in a comment somewhere for more context.
There was a problem hiding this comment.
Done. I hadn't checked before but it really is a regression in 1.21.
Issue go-delve#3548 describes a bug in the compiler which was fixed by commit 505e50b. But this case wasn't covered by our current tests (obviously) and the fix in the compiler looks accidental so it's worth adding a test for it. Fixes go-delve#3548
kakkoyun
referenced
this pull request
in parca-dev/parca-agent
Jan 8, 2024
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/go-delve/delve](https://togithub.com/go-delve/delve) | `v1.21.2` -> `v1.22.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>go-delve/delve (github.com/go-delve/delve)</summary> ### [`v1.22.0`](https://togithub.com/go-delve/delve/releases/tag/v1.22.0) [Compare Source](https://togithub.com/go-delve/delve/compare/v1.21.2...v1.22.0) #### What's Changed - all: remove obsolete build tags by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3513](https://togithub.com/go-delve/delve/pull/3513) - service/dap: add the concrete type to interface type by [@​suzmue](https://togithub.com/suzmue) in [https://github.com/go-delve/delve/pull/3510](https://togithub.com/go-delve/delve/pull/3510) - service/dap: fix bugs in stdout/stderr handling by [@​hyangah](https://togithub.com/hyangah) in [https://github.com/go-delve/delve/pull/3522](https://togithub.com/go-delve/delve/pull/3522) - pkg/terminal: support vscode in edit command by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3524](https://togithub.com/go-delve/delve/pull/3524) - Use a valid timezone in tested binary by [@​upils](https://togithub.com/upils) in [https://github.com/go-delve/delve/pull/3527](https://togithub.com/go-delve/delve/pull/3527) - proc: implement follow exec mode on Windows by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3507](https://togithub.com/go-delve/delve/pull/3507) - service/test: disable TestClientServer_chanGoroutines with rr backend by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3492](https://togithub.com/go-delve/delve/pull/3492) - proc/native: use cgo instead of C for freebsd by [@​4a6f656c](https://togithub.com/4a6f656c) in [https://github.com/go-delve/delve/pull/3529](https://togithub.com/go-delve/delve/pull/3529) - proc: use DW_AT_trampoline to detect auto-generated code by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3528](https://togithub.com/go-delve/delve/pull/3528) - proc: use stack machine to evaluate expressions by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3508](https://togithub.com/go-delve/delve/pull/3508) - proc: fix comment typos by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3531](https://togithub.com/go-delve/delve/pull/3531) - proc: add min and max builtins by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3530](https://togithub.com/go-delve/delve/pull/3530) - CI: update staticcheck version by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3533](https://togithub.com/go-delve/delve/pull/3533) - proc: remove expr evaluation goroutine from EvalExpressionWithCalls by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3532](https://togithub.com/go-delve/delve/pull/3532) - service/api: use bitfield for prettyprint flags by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3536](https://togithub.com/go-delve/delve/pull/3536) - proc: allow evaluator to reference previous frames by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3534](https://togithub.com/go-delve/delve/pull/3534) - \*: Add explicit code of conduct by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3540](https://togithub.com/go-delve/delve/pull/3540) - proc,service/dap,proc/gdbserial: fixes for debugserver --unmask-signals by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3541](https://togithub.com/go-delve/delve/pull/3541) - \*: release 1.21.2 (for master) by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3546](https://togithub.com/go-delve/delve/pull/3546) - proc/native: wherever we check for exited we should also check detached by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3547](https://togithub.com/go-delve/delve/pull/3547) - pkg/proc: add inline function support for stripped binaries by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3549](https://togithub.com/go-delve/delve/pull/3549) - cmd: fix a bunch of linter warnings from GoLand by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3550](https://togithub.com/go-delve/delve/pull/3550) - proc: add regression test for issue [#​3548](https://togithub.com/go-delve/delve/issues/3548) by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3553](https://togithub.com/go-delve/delve/pull/3553) - \*: update dependencies by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3552](https://togithub.com/go-delve/delve/pull/3552) - service: fix a bunch of linter warnings from GoLand by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3551](https://togithub.com/go-delve/delve/pull/3551) - \*: Correct spelling mistakes by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3555](https://togithub.com/go-delve/delve/pull/3555) - pkg,service: Remove redundant build constraints by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3556](https://togithub.com/go-delve/delve/pull/3556) - Shorten variable types by [@​stefanhaller](https://togithub.com/stefanhaller) in [https://github.com/go-delve/delve/pull/3535](https://togithub.com/go-delve/delve/pull/3535) - TeamCity: reupgrade linux/386 builder to Go 1.21 by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3560](https://togithub.com/go-delve/delve/pull/3560) - pkg/proc: improve support unwinding from sigpanic by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3559](https://togithub.com/go-delve/delve/pull/3559) - pkg/proc: unskip passing tests and reorganize by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3561](https://togithub.com/go-delve/delve/pull/3561) - chore: use strings.Contains instead by [@​testwill](https://togithub.com/testwill) in [https://github.com/go-delve/delve/pull/3562](https://togithub.com/go-delve/delve/pull/3562) - pkg,service: remove unnecessary convertions by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3564](https://togithub.com/go-delve/delve/pull/3564) - \*: remove checks for TRAVIS env variable by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3568](https://togithub.com/go-delve/delve/pull/3568) - terminal: clear substitute path rules cache when config is used by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3567](https://togithub.com/go-delve/delve/pull/3567) - service: fix typo in variable name by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3575](https://togithub.com/go-delve/delve/pull/3575) - TeamCity: remove windows/arm64 builders from chain by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3572](https://togithub.com/go-delve/delve/pull/3572) - proc: simplify and generalize runtime.mallocgc workaround by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3571](https://togithub.com/go-delve/delve/pull/3571) - proc/gdbserial: refactor parsing of key-value pairs from gdb protocol by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3574](https://togithub.com/go-delve/delve/pull/3574) - Extract tip builds into a separate sub-project by [@​artspb](https://togithub.com/artspb) in [https://github.com/go-delve/delve/pull/3578](https://togithub.com/go-delve/delve/pull/3578) - pkg,service/dap: use switch instead of if-else-if by [@​alexandear](https://togithub.com/alexandear) in [https://github.com/go-delve/delve/pull/3576](https://togithub.com/go-delve/delve/pull/3576) - CI: update teamcity settings by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3579](https://togithub.com/go-delve/delve/pull/3579) - pkg/proc: use gore to obtain info from stripped binaries by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3577](https://togithub.com/go-delve/delve/pull/3577) - add missing import in TeamCity DSL by [@​artspb](https://togithub.com/artspb) in [https://github.com/go-delve/delve/pull/3580](https://togithub.com/go-delve/delve/pull/3580) - update TeamCity DSL version to 2023.05 and remove tip configurations from Aggregator by [@​artspb](https://togithub.com/artspb) in [https://github.com/go-delve/delve/pull/3581](https://togithub.com/go-delve/delve/pull/3581) - proc: fix TestIssue1101 flake by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3585](https://togithub.com/go-delve/delve/pull/3585) - proc: skip trapthread for harcoded breakpoints after manual stop by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3582](https://togithub.com/go-delve/delve/pull/3582) - tests: fix tests in go1.22 by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3583](https://togithub.com/go-delve/delve/pull/3583) - all: run go mod tidy by [@​prattmic](https://togithub.com/prattmic) in [https://github.com/go-delve/delve/pull/3589](https://togithub.com/go-delve/delve/pull/3589) - add all branches but PRs to filter by [@​artspb](https://togithub.com/artspb) in [https://github.com/go-delve/delve/pull/3590](https://togithub.com/go-delve/delve/pull/3590) - service/dap: fix close on closed channel panic by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3573](https://togithub.com/go-delve/delve/pull/3573) - Show pprof labels in thread names by [@​stefanhaller](https://togithub.com/stefanhaller) in [https://github.com/go-delve/delve/pull/3501](https://togithub.com/go-delve/delve/pull/3501) - \*: Use forked goretk/gore module by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3597](https://togithub.com/go-delve/delve/pull/3597) - proc: make some type casts less counterintuitive by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3596](https://togithub.com/go-delve/delve/pull/3596) - teamcity,version: add 1.22 to supported versions and CI matrix by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3601](https://togithub.com/go-delve/delve/pull/3601) - proc: fix ppc64 arch name check by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3608](https://togithub.com/go-delve/delve/pull/3608) - goversion: include pre-releases in VersionAfterOrEqual check by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3607](https://togithub.com/go-delve/delve/pull/3607) - service/dap: fix close on closed channel by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3609](https://togithub.com/go-delve/delve/pull/3609) - proc: fix TestPackageRenames on go1.22 on linux/386 by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3610](https://togithub.com/go-delve/delve/pull/3610) - \*: Update gore dep for 1.22 by [@​derekparker](https://togithub.com/derekparker) in [https://github.com/go-delve/delve/pull/3611](https://togithub.com/go-delve/delve/pull/3611) - \*: release version 1.22.0 by [@​aarzilli](https://togithub.com/aarzilli) in [https://github.com/go-delve/delve/pull/3606](https://togithub.com/go-delve/delve/pull/3606) #### New Contributors - [@​upils](https://togithub.com/upils) made their first contribution in [https://github.com/go-delve/delve/pull/3527](https://togithub.com/go-delve/delve/pull/3527) - [@​testwill](https://togithub.com/testwill) made their first contribution in [https://github.com/go-delve/delve/pull/3562](https://togithub.com/go-delve/delve/pull/3562) - [@​prattmic](https://togithub.com/prattmic) made their first contribution in [https://github.com/go-delve/delve/pull/3589](https://togithub.com/go-delve/delve/pull/3589) **Full Changelog**: go-delve/delve@v1.21.1...v1.22.0 **Curated Changelog**: https://github.com/go-delve/delve/blob/master/CHANGELOG.md#1220-2023-12-29 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/parca-dev/parca-agent). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMjEuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEyMS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->
](https://renovatebot.com){kind=link}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #3548 describes a bug in the compiler which was fixed by commit
505e50b. But this case wasn't covered by our current tests (obviously)
and the fix in the compiler looks accidental so it's worth adding a
test for it.
Fixes #3548