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

Update to nightly-2023-11-25 #14

Closed
wants to merge 1 commit into from

Conversation

bjorn3
Copy link

@bjorn3 bjorn3 commented Nov 28, 2023

No description provided.

@@ -650,7 +654,8 @@ fn place_projection_conflict<'tcx>(
| ProjectionElem::ConstantIndex { .. }
| ProjectionElem::OpaqueCast { .. }
| ProjectionElem::Subslice { .. }
| ProjectionElem::Downcast(..),
| ProjectionElem::Downcast(..)
| ProjectionElem::Subtype(..),
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if the changes in this file are correct.

@bjorn3
Copy link
Author

bjorn3 commented Nov 28, 2023

Locally I'm getting a test failure which looks like UB or a miscompilation somewhere. Basically for the arg test the output is empty, but if I change the println!() to eprintln!() it does show up on stderr. Furthermore if I eprintln!("{:?}", args), allcaps is shown as true, but if I remove the if args.allcaps { msg = msg.to_uppercase(); }, it suddently changes to false.

@willcrichton
Copy link
Contributor

Thanks for the PR. (Just curious, are you using rustc_plugin for something?)

I will handle bumping the nightly (see #16). I decided to delete the entire places_conflict module because the new rustc_borrowck::consumers module now provides the same functionality publicly: https://doc.rust-lang.org/nightly/nightly-rustc/rustc_borrowck/consumers/fn.places_conflict.html

@bjorn3
Copy link
Author

bjorn3 commented Dec 6, 2023

I was looking into updating flowistry to the latest rustc version. I have been working on spme signficant refactors to the driver interface and wanted to make sure I wouldn't break flowistry.

@willcrichton
Copy link
Contributor

Sorry I let this drop, but I'll just add -- @bjorn3 don't worry about breaking Flowistry, I am very used to it breaking every time the compiler updates 🙃 a changelog is more than enough!

@bjorn3
Copy link
Author

bjorn3 commented Jan 7, 2024

In this case it is a non-trivial change of the rustc_driver interface that I fear will break flowistry in a way that could potentially require a rustc change to fix it again. To be precise I want to get rid of the rustc_driver query interface.

@bjorn3 bjorn3 closed this Jan 7, 2024
@bjorn3 bjorn3 deleted the rustup branch January 7, 2024 20:47
@willcrichton
Copy link
Contributor

Got it. If you want to test anything, I just updated the dev branch of Flowistry to nightly-2024-01-06.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants