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
cmd/compile/internal/syntax: needs regression test framework #20800
Comments
Marking for 1.10 simply so it stays on the radar. |
Test case for #20789 (https://go-review.googlesource.com/#/c/46713/) could move there. |
Test case for #23434 (https://go-review.googlesource.com/c/go/+/87898, corresponding go/parser fix) should be added here. |
Test case for #23385 should be added here. |
Change https://golang.org/cl/88335 mentions this issue: |
Change https://golang.org/cl/88336 mentions this issue: |
R=go1.11. Now that we have a syntax error test harness, we can add the proper tests for the recent parser fixes. For #20800. For #20789. For #23385. For #23434. A test for #20789 already exists in test/fixedbugs, but this is the better location for that test. But leaving the existing one where it is as well. Change-Id: I5937b9b63bafd1efab467a00344302e717976171 Reviewed-on: https://go-review.googlesource.com/88336 Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Reminder issue: The syntax package doesn't have a good setup for adding regression tests.
The text was updated successfully, but these errors were encountered: