-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
[WIP] polonius: adapt to the new fact format #68993
Conversation
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors delegate=albins r=me but travis is unhappy. @AlbinS, I've delegated bors to you, so if you are able to fix these errors you should be able to tell bors |
✌️ @AlbinS can now approve this pull request |
@nikomatsakis Thanks! It needs a new Polonius release (the PR I’m working on). |
Blocked waiting on rust-lang/polonius#135 |
rust-lang/polonius#135 is now merged, and |
Since Albin won't be able to do it for a while, I've rebased and finalized this PR in #69482, so I'll close this one. |
Polonius: update `polonius-engine` to 0.12.0 Since @AlbinS won't have the time to finish up #68993 for a while, I'll take care of the trivial remaining tasks (rebasing, taking care of tidy/rustfmt). I'll r? @nikomatsakis since they're assigned to #68993, but have actually [already reviewed it pre-rebase](#68993 (comment)). When CI passes: I'll notify bors and close #68993, since this PR supersedes it.
Hopefully the final phase of the ordeal now known as The Great Renaming, in which the Polonius working group renames virtually all of their input facts.
This also extends fact generation to support full initialisation tracking and move analysis in Polonius.