Skip to content

Conversation

@bjorn3
Copy link
Member

@bjorn3 bjorn3 commented Nov 28, 2025

This way we don't need a special case in MiriBeRustCompilerCalls.

This way we don't need a special case in MiriBeRustCompilerCalls.
@rustbot
Copy link
Collaborator

rustbot commented Nov 28, 2025

Thank you for contributing to Miri! A reviewer will take a look at your PR, typically within a week or two.
Please remember to not force-push to the PR branch except when you need to rebase due to a conflict or when the reviewer asks you for it.

@rustbot rustbot added the S-waiting-on-review Status: Waiting for a review to complete label Nov 28, 2025
Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

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

Good call, I like the general approach. We can also update a bunch of comments and names then, I think.

View changes since this review

src/bin/miri.rs Outdated
// builds, but we are setting `-Zalways-encode-mir` which avoids those issues.
let _ = tcx.collect_and_partition_mono_items(());
}
// cargo-miri has patched the compiler flags to make these into check-only builds,
Copy link
Member

Choose a reason for hiding this comment

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

That's not even correct any more, is it? You removed that flag patching in #4648.

Copy link
Member Author

Choose a reason for hiding this comment

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

The call is still necessary here as DummyCodegenBackend doesn't have it. I will update the comment though.

@rustbot rustbot added S-waiting-on-author Status: Waiting for the PR author to address review comments and removed S-waiting-on-review Status: Waiting for a review to complete labels Nov 28, 2025
Co-Authored-By: Ralf Jung <post@ralfj.de>
@RalfJung
Copy link
Member

That's annoying that we can't use return to make it clear that the call won't return... is there some other nice way to make that syntactically obvious?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: Waiting for the PR author to address review comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants