-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Split rustc into libraries #560
Comments
syntax and rustc (middle-and-back) are their own libraries now. I think this is an ok split. closing. |
there hasn't actually been a library split yet, just some reorganization |
The rustsyntax crate now contains the parser, pretty-printer and syntax extensions. Probably syntax extensions belong somewhere else eventually. The metadata module is also a good candidate for extracting into its own library, since it will be needed for dynamic loading and other reflection purposes. The few frontend passes that remain mostly do extension-like things and probably belong somewhere else eventually. The directory structure of rustc is a bit weird right now. Technically, rustc is just one .rs file and everything under src/rustc should be under src/librustc. |
Things are under src/librustc now. |
@sanxiyn Though the division of libraries is the same as of brson's last comment. I think that splitting librustc into more libs would be nice, if only because a small number of big chunks are easier to understand than a single enormous chunk. |
I don't believe this is backwards incompatible, renominating. |
just a bug, removing milestone/nomination. |
Still relevant |
triage, no changes |
Visiting for triage. No change. |
Might as well close this. Everybody knows rustc needs to be refactored. |
The official name of the memory intrinsics has changed to `memory.size` and `memory.grow`, so let's reflect that with our naming as well! Additionally they have an argument of which memory to operate on with LLVM and must always be zero currently.
Remove unused link references.
rust-lang#560 (rust-lang#952) Add tests for a few open issues.
ci: bump actions/checkout to v4
Parts of rustc will need to be used outside of the compiler and splitting it up will help improve the design.
One possible division:
creader needs to be somewhere as well. Needs more thought and discussion.
The text was updated successfully, but these errors were encountered: