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

x/tools/go/ssa/interp: tests broken at Go CL 191198 #34089

Open
bcmills opened this issue Sep 4, 2019 · 4 comments
Open

x/tools/go/ssa/interp: tests broken at Go CL 191198 #34089

bcmills opened this issue Sep 4, 2019 · 4 comments

Comments

@bcmills
Copy link
Member

@bcmills bcmills commented Sep 4, 2019

CL 191198 changed the error message written by the compiler for unhashable types.

Apparently the tests for x/tools/go/ssa/interp were relying on the exact error message, because they are now failing in the builders:
https://build.golang.org/log/b4fad15a40867a4f25fdf2fdacbe3bcc13408bad

CC @ianthehat @randall77 @martisch

@gopherbot gopherbot added this to the Unreleased milestone Sep 4, 2019
@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Sep 4, 2019

Change https://golang.org/cl/193377 mentions this issue: go/ssa/interp: skip recover2 testcase

gopherbot pushed a commit to golang/tools that referenced this issue Sep 4, 2019
The test case (taken from the go distribution itself) seems to be broken
when run against tip. Disable it when run against tip.

Updates golang/go#34089

Change-Id: I47d2e12ca39bc060fe0a0c967e95b900bdd0d725
Reviewed-on: https://go-review.googlesource.com/c/tools/+/193377
Run-TryBot: Michael Matloob <matloob@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
@stamblerre stamblerre added the Tools label Sep 9, 2019
@bcmills bcmills added Soon and removed Soon labels Sep 10, 2019
@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Feb 25, 2020

Change https://golang.org/cl/220957 mentions this issue: go/ssa: disable recover2 testcase

gopherbot pushed a commit to golang/tools that referenced this issue Feb 25, 2020
We still haven't figured out what's going on, but this is a low
priority because the SSA interpreter isn't used by anyone.
The primary concern is that this might be covering a legitimate
underlying SSA bug.

Updates golang/go#34089

Change-Id: I2886845b4f4b011104de945d04d6b8c4c436cbe9
Reviewed-on: https://go-review.googlesource.com/c/tools/+/220957
Run-TryBot: Michael Matloob <matloob@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
@matloob matloob removed the Soon label Feb 25, 2020
@matloob

This comment has been minimized.

Copy link
Contributor

@matloob matloob commented Feb 25, 2020

Removing Soon because ssa/interp is not heavily used, and the test is no longer a time bomb.

@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Feb 25, 2020

Change https://golang.org/cl/220959 mentions this issue: go/ssa/interp: remove unused import

gopherbot pushed a commit to golang/tools that referenced this issue Feb 25, 2020
Updates golang/go#34089

Change-Id: Ic7cc45803a31394241c7fa33092ec566dbf296ca
Reviewed-on: https://go-review.googlesource.com/c/tools/+/220959
Run-TryBot: Jay Conrod <jayconrod@google.com>
Reviewed-by: Michael Matloob <matloob@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.