Skip to content

make liballoc internal test suite mostly pass in Miri #60077

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

Merged
merged 2 commits into from
Apr 19, 2019

Conversation

RalfJung
Copy link
Member

I discovered, to my surprise, that liballoc has two test suites: liballoc/tests, and a bunch of stuff embedded directly within liballoc. The latter was not covered by miri-test-libstd yet. This disables in Miri the tests that Miri cannot support or runs too slowly.

@rust-highfive
Copy link
Contributor

r? @joshtriplett

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 18, 2019
@joshtriplett
Copy link
Member

Made one quick comment about the placement of a declaration, r=me with that fixed.

@RalfJung
Copy link
Member Author

Done.

@bors r=joshtriplett rollup

@bors
Copy link
Collaborator

bors commented Apr 19, 2019

📌 Commit 2bc8c54 has been approved by joshtriplett

@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-review Status: Awaiting review from the assignee but also interested parties. labels Apr 19, 2019
@bors
Copy link
Collaborator

bors commented Apr 19, 2019

⌛ Testing commit 2bc8c54 with merge a2bbf7d...

bors added a commit that referenced this pull request Apr 19, 2019
make liballoc internal test suite mostly pass in Miri

I discovered, to my surprise, that liballoc has two test suites: `liballoc/tests`, and a bunch of stuff embedded directly within liballoc.  The latter was not covered by [miri-test-libstd](https://github.com/RalfJung/miri-test-libstd) yet.  This disables in Miri the tests that Miri cannot support or runs too slowly.
@bors
Copy link
Collaborator

bors commented Apr 19, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: joshtriplett
Pushing a2bbf7d to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 19, 2019
@bors bors merged commit 2bc8c54 into rust-lang:master Apr 19, 2019
@RalfJung RalfJung deleted the miri-alloc-tests branch June 10, 2019 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants