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.

Sign up for GitHub

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

Jump to bottom

Store types in a side table #332

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

Store types in a side table #332

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

Comments

@pcwalton
Copy link
Contributor

We should store types in a side table instead of rebuilding the AST during typeck. I don't think this will be as much of a perf win as interning types will be, but it should be significant.
@graydon
Copy link
Contributor

graydon commented May 26, 2011

This is done now, yes?

@pcwalton
Copy link
Contributor Author

Yes.

oli-obk pushed a commit to oli-obk/rust that referenced this issue Sep 19, 2017
@RalfJung
Merge pull request rust-lang#332 from solson/rustup
8307d27 
rustup
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
@alexcrichton
Merge pull request rust-lang#332 from cuviper/solaris-rtld
b0d6253 
solaris: add dlsym/dlopen RTLD constants
Aaron1011 pushed a commit to Aaron1011/rust that referenced this issue Oct 26, 2020
@bjorn3
Merge pull request rust-lang#332 from bjorn3/dependabot/cargo/rand_ji…
7b1f052 
…tter-0.1.3

Bump rand_jitter from 0.1.2 to 0.1.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-type-system Area: Type system
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@graydon @pcwalton