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

Intern types #330

Closed
pcwalton opened this issue Apr 16, 2011 · 2 comments
Closed

Intern types #330

pcwalton opened this issue Apr 16, 2011 · 2 comments
Labels
A-type-system Area: Type system

Comments

@pcwalton
Copy link
Contributor

rustc is unbearably slow at the moment, largely due to unify. We should do what LLVM does and intern types in one global table and refer to them by index into that table. We can then keep a cache on the side of unified types.

@graydon
Copy link
Contributor

graydon commented May 26, 2011

This is done, yes?

@pcwalton
Copy link
Contributor Author

Yup.

On Thu, May 26, 2011 at 2:12 PM, graydon
reply@reply.github.com
wrote:

This is done, yes?

Reply to this email directly or view it on GitHub:
https://github.com/graydon/rust/issues/330#comment_1244636

oli-obk pushed a commit to oli-obk/rust that referenced this issue Sep 19, 2017
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
kazcw pushed a commit to kazcw/rust that referenced this issue Oct 23, 2018
This seems likely to have mostly just been a copy/paste error, so this
re-reviews the intrinsics and aligns it with the implementation in
clang.

Closes rust-lang#328
dlrobertson pushed a commit to dlrobertson/rust that referenced this issue Nov 29, 2018
Fix the definition of MetaItem and MetaSeq
Aaron1011 pushed a commit to Aaron1011/rust that referenced this issue Oct 26, 2020
antoyo added a commit to antoyo/rust that referenced this issue Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-type-system Area: Type system
Projects
None yet
Development

No branches or pull requests

2 participants