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

cmd/compile/internal/importer,go/internal/gcimporter: remove iimport.go export data importers #68616

Closed
timothy-king opened this issue Jul 26, 2024 · 2 comments
Assignees
Labels
compiler/runtime Issues related to the Go compiler and/or runtime.
Milestone

Comments

@timothy-king
Copy link
Contributor

Tracking bug for removing the iimport.go files for the iexport data format in the compiler.

The unified goexperiment was removed in 1.21. Once #64751 happens, the bootstrap compiler will be above 1.21, I am not aware of a reason these importers could be use in the compiler.

x/tools will still have a copy for gopls, which creates this format, and support for building tools that support export data created by older compilers.

@timothy-king timothy-king added the compiler/runtime Issues related to the Go compiler and/or runtime. label Jul 26, 2024
@timothy-king timothy-king added this to the Go1.24 milestone Jul 26, 2024
@timothy-king timothy-king self-assigned this Jul 26, 2024
@gabyhelp
Copy link

Related Issues and Documentation

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

@timothy-king
Copy link
Contributor Author

Closing this issue. I am going to think through the implications of removing go/internal/gcimporter/iimport.go support a bit more. I'll open a new issue when I have reached a conclusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler/runtime Issues related to the Go compiler and/or runtime.
Projects
None yet
Development

No branches or pull requests

2 participants