Skip to content

Conversation

@folkertdev
Copy link
Contributor

fixes #150050

Always use the same implementation when running miri (regardless of target, for consistency). This implementation yields the current thread to try find actual bugs.

r? RalfJung

@rustbot
Copy link
Collaborator

rustbot commented Dec 17, 2025

The Miri subtree was changed

cc @rust-lang/miri

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Dec 17, 2025
@rust-log-analyzer

This comment has been minimized.

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.

Thanks! Just some minor suggestions.

View changes since this review

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 17, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 17, 2025

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@folkertdev folkertdev force-pushed the riscv-pause-miri branch 2 times, most recently from eef6c46 to c3be846 Compare December 17, 2025 14:48
@RalfJung
Copy link
Member

Thanks, LGTM! r=me when CI is green.
@bors delegate+

@bors
Copy link
Collaborator

bors commented Dec 17, 2025

✌️ @folkertdev, you can now approve this pull request!

If @RalfJung told you to "r=me" after making some further change, please make that change, then do @bors r=@RalfJung

@folkertdev
Copy link
Contributor Author

@bors r=RalfJung

@bors
Copy link
Collaborator

bors commented Dec 17, 2025

📌 Commit c7dd19b has been approved by RalfJung

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 17, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Dec 17, 2025
…fJung

miri: add `miri_spin_loop` to make `hint::spin_loop` work consistently

fixes rust-lang#150050

Always use the same implementation when running miri (regardless of target, for consistency). This implementation yields the current thread to try find actual bugs.

r? RalfJung
bors added a commit that referenced this pull request Dec 17, 2025
Rollup of 4 pull requests

Successful merges:

 - #149919 (Correctly encode doc attribute metadata)
 - #150051 (mir_build: Rename `TestCase` to `TestableCase`)
 - #150088 (miri: add `miri_spin_loop` to make `hint::spin_loop` work consistently)
 - #150096 (Revert #148937)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 15c54bc into rust-lang:main Dec 17, 2025
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Dec 17, 2025
rust-timer added a commit that referenced this pull request Dec 17, 2025
Rollup merge of #150088 - folkertdev:riscv-pause-miri, r=RalfJung

miri: add `miri_spin_loop` to make `hint::spin_loop` work consistently

fixes #150050

Always use the same implementation when running miri (regardless of target, for consistency). This implementation yields the current thread to try find actual bugs.

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

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

riscv64: miri test: error: unsupported operation: inline assembly is not supported

5 participants