-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Try caching macro calls more aggressively in Semantics #17004
Conversation
☔ The latest upstream changes (presumably #17092) made this pull request unmergeable. Please resolve the merge conflicts. |
ae62c8c
to
f55e74e
Compare
1b5d608
to
a3b3127
Compare
a3b3127
to
7d40763
Compare
8ca2fe0
to
9349045
Compare
@bors r+ |
☀️ Test successful - checks-actions |
@Veykril dumb question: why is this cached outside of salsa? Do you have a good rule of thumb for deciding which state should be in salsa versus in the struct directly like this example? |
Most of the state here depends on syntax trees, those have no business being inside salsa because they are heavy memory wise and very unstable. |
No description provided.