-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Use a single CtxtInterners #61392
Use a single CtxtInterners #61392
Conversation
@bors try |
⌛ Trying commit bc52e4f1457cf7594dd3f4cd034b55a20a247068 with merge 870e5701636f71457ab8ee1a43db9da4e602c0c2... |
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 |
☀️ Try build successful - checks-travis |
@rust-timer build 870e5701636f71457ab8ee1a43db9da4e602c0c2 |
Success: Queued 870e5701636f71457ab8ee1a43db9da4e602c0c2 with parent 3ade426, comparison URL. |
Finished benchmarking try commit 870e5701636f71457ab8ee1a43db9da4e602c0c2, comparison URL. |
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 |
☔ The latest upstream changes (presumably #61418) made this pull request unmergeable. Please resolve the merge conflicts. |
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 |
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 |
☔ The latest upstream changes (presumably #61817) made this pull request unmergeable. Please resolve the merge conflicts. |
48a01ba
to
fc4d492
Compare
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 |
This includes #61871. |
☔ The latest upstream changes (presumably #61945) made this pull request unmergeable. Please resolve the merge conflicts. |
Rebased and ready for review. |
@@ -1015,8 +974,6 @@ pub struct FreeRegionInfo { | |||
#[derive(Copy, Clone)] | |||
pub struct TyCtxt<'tcx> { | |||
gcx: &'tcx GlobalCtxt<'tcx>, |
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.
Oh we should probably get rid of gcx
as well, heh.
@bors r+ |
@bors r=eddyb |
📌 Commit ec8c2e1 has been approved by |
Use a single CtxtInterners Builds on rust-lang#57214 r? @eddyb
Use a single CtxtInterners Builds on rust-lang#57214 r? @eddyb
Use a single CtxtInterners Builds on rust-lang#57214 r? @eddyb
☀️ Test successful - checks-azure, checks-travis, status-appveyor |
Builds on #57214
r? @eddyb