-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Use the new "durability" infrastructure from salsa #1445
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
Conversation
Some numbers:
previously, what we were doing was basically |
eprint!("comment change: "); | ||
{ | ||
let mut text = host.analysis().file_text(file_id).to_string(); | ||
text.push_str("\n/* Hello world */\n"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a slightly more interesting change might be putting the comment at the top?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be harder to implement, because I would need to adjust offset for subsequent queries.
ideally, we should accept a .patch
file as a command line argument...
61b0c1c
to
b3d791d
Compare
4086608
to
3b312b4
Compare
a4f06ba
to
bdd9b11
Compare
bors r+ |
1445: Use the new "durability" infrastructure from salsa r=matklad a=matklad Based on https://github.com/nikomatsakis/salsa/tree/durability Durability allows us to skip *validation* work for sysroot and crates.io libraries, which massively speeds up some workloads Co-authored-by: Aleksey Kladov <[email protected]>
Build succeeded |
Based on https://github.com/nikomatsakis/salsa/tree/durability
Durability allows us to skip validation work for sysroot and crates.io libraries, which massively speeds up some workloads