-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Add method to retrieve body of closure in stable-mir #141307
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
Conversation
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.
Awesome! If it's not too much work, can you please add a test for a closure that takes a generic argument?
This comment has been minimized.
This comment has been minimized.
f795c8d
to
363ae39
Compare
Sorry for addressing your comments so late. I have added the test with the generic closure and a doc comment (though I'm not sure it offers much information). |
This comment has been minimized.
This comment has been minimized.
363ae39
to
8604e58
Compare
Thanks! @bors r+ rollup |
Rollup of 10 pull requests Successful merges: - #141307 (Add method to retrieve body of closure in stable-mir) - #142040 (transmutability: shift abstraction boundary) - #142066 (More simple 2015 edition test decoupling) - #142157 (rustc_resolve: Improve `resolve_const_param_in_non_trivial_anon_const` wording) - #142217 (`tests/ui`: A New Order [10/N]) - #142219 (`tests/ui`: A New Order [11/N]) - #142261 (use correct edition when warning for unsafe attributes) - #142303 (Assorted bootstrap cleanups (step 1)) - #142318 (Cleanup `rust-src` remapping and real dir) - #142352 (compiler: Make `c_int_width` an integer) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of #141307 - b-naber:closure-body, r=celinval Add method to retrieve body of closure in stable-mir fixes rust-lang/project-stable-mir#85 r? `@celinval`
fixes rust-lang/project-stable-mir#85
r? @celinval