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/internal/gcimporter: update iexport.go to support generics #48392

findleyr opened this issue Sep 14, 2021 · 1 comment

x/tools/internal/gcimporter: update iexport.go to support generics #48392

findleyr opened this issue Sep 14, 2021 · 1 comment


Copy link

@findleyr findleyr commented Sep 14, 2021

CL 349949 updates x/tools/internal/gcimporter/iimport.go to support generics, by porting over changes from the standard library go/internal/gcimporter package. It will take longer to port iexport.go, as it must be translated from cmd/compile/internal/typecheck/iexport.go. In the meantime (~days, I expect), I think we should skip x/tools/go/internal/gcimporter.TestIExportData_stdlib on Go 1.18, in order to avoid failures on the x/tools Trybots on Go repo CLs that use generics.

This issue tracks that process (skipping the test, updating the exporter, and unskipping).

CC @griesemer @mdempsky @danscales @ianlancetaylor

Copy link

@gopherbot gopherbot commented Sep 14, 2021

Change mentions this issue: go/internal/gcimporter: skip TestIExportData_stdlib on go1.18

gopherbot pushed a commit to golang/tools that referenced this issue Sep 14, 2021
This test will fail on Go 1.18 once standard library changes are
introduced that use generics. Skip it until iexport.go has been updated
to support generics.

Updates golang/go#48392

Change-Id: I0f779689daaf2bc3127affff633c883ed9d7289b
Trust: Robert Findley <>
Run-TryBot: Robert Findley <>
Reviewed-by: Robert Griesemer <>
gopls-CI: kokoro <>
TryBot-Result: Go Bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants