-
Notifications
You must be signed in to change notification settings - Fork 89
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
Upgrade rust toolchain to 2022-11-20 #1927
Upgrade rust toolchain to 2022-11-20 #1927
Conversation
ae253dc
to
f3a4997
Compare
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.
Thanks for doing this. I have a couple of questions.
@@ -176,7 +176,9 @@ impl<'tcx> ProjectedPlace<'tcx> { | |||
goto_expr, expr_ty, ty_from_mir | |||
); | |||
warn!("{}", msg); | |||
debug_assert!(false, "{}", msg); | |||
// TODO: this assertion fails on firecracker with the rust 2022-11-20 toolchain |
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.
😕 Do you know why?
@@ -53,6 +53,7 @@ impl<'tcx> GotocCtx<'tcx> { | |||
self.current_fn().mir().var_debug_info.iter().find(|info| match info.value { | |||
VarDebugInfoContents::Place(p) => p.local == *l && p.projection.len() == 0, | |||
VarDebugInfoContents::Const(_) => false, | |||
VarDebugInfoContents::Composite { .. } => todo!(), |
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.
Can you please create an issue and add more details there? I wonder if we should really crash in this case.
e33a5f0
to
fe84d4c
Compare
Description of changes:
Upgrade rust toolchain to 2022-11-20.
Resolved issues:
Resolves #1923
Related RFC:
Optional #ISSUE-NUMBER.
Call-outs:
This causes a debug_assert in
ProjectPlace::try_new
to fail. I've disabled the debug assert which will make codegen produce acodegen_unimplemented
.Testing:
How is this change tested? Existing regressions
Is this a refactor change? No
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.