-
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
Support debuginfo for custom MIR. #115540
Conversation
(rustbot has picked a reviewer for you, use r? to override) |
r? compiler |
@bors r+ rollup |
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#113510 (Document soundness of Integer -> Pointer -> Integer conversions in `const` contexts.) - rust-lang#114412 (document our assumptions about symbols provided by the libc) - rust-lang#114813 (explain why we can mutate the FPU control word) - rust-lang#115523 (improve `AttrTokenStream`) - rust-lang#115536 (interpret: make MemPlace, Place, Operand types private to the interpreter) - rust-lang#115540 (Support debuginfo for custom MIR.) - rust-lang#115563 (llvm-wrapper: adapt for LLVM API change) r? `@ghost` `@rustbot` modify labels: rollup
Did this break stage 0 Miri tests?
On CI we only run stage 2 tests, but for local development stage 0 tests are very valuable. |
@@ -371,6 +379,7 @@ pub macro mir { | |||
( | |||
$(type RET = $ret_ty:ty ;)? | |||
$(let $local_decl:ident $(: $local_decl_ty:ty)? ;)* | |||
$(debug $dbg_name:ident => $dbg_data:expr ;)* |
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.
Would be great if some doc comment somewhere explained this syntax.
No description provided.