Skip to content
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.

Sign up for GitHub

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

Jump to bottom

use intptrcast for heap_allocator test; then it should work on Windows #809

Merged
merged 7 commits into from
Jun 30, 2019
Merged

use intptrcast for heap_allocator test; then it should work on Windows #809

merged 7 commits into from
Jun 30, 2019

Conversation

RalfJung
Copy link
Member

No description provided.

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

📌 Commit b067313 has been approved by RalfJung

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

📌 Commit 0ea4b50 has been approved by RalfJung

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Testing commit 0ea4b50 with merge b0af195...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=RalfJung
b0af195 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

💔 Test failed - status-appveyor

@RalfJung
Copy link
Member Author

Well that's odd...

error[E0080]: Miri evaluation error: a raw memory access tried to access part of a pointer value as raw bytes
  --> $DIR/heap_allocator.rs:43:24
   |
43 |             assert_eq!((ptr.as_ptr() as usize) % align, 0,
   |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Miri evaluation error: a raw memory access tried to access part of a pointer value as raw bytes

So calling the allocation function actually succeeds, but then later checking alignment does not.

@RalfJung
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Trying commit 1f945cc with merge 58df88f...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=<try>
58df88f 
use intptrcast for heap_allocator test; then it should work on Windows
@RalfJung
Copy link
Member Author

Wow, MIRI_BACKTRACE makes everything just crazy slow, at least on Windows.

@oli-obk this build log looks like we are throwing and catching tons of Miri errors. That doesn't seem right.

@RalfJung
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Trying commit 2a92157 with merge 899e5bf...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=<try>
899e5bf 
use intptrcast for heap_allocator test; then it should work on Windows
@RalfJung
Copy link
Member Author

D'oh, I've failed to actually set the compiler flag correctly...

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

📌 Commit 78261b7 has been approved by RalfJung

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Testing commit 78261b7 with merge bb45bdb...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=RalfJung
bb45bdb 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

💔 Test failed - status-appveyor

@RalfJung
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Trying commit 20753d9 with merge 2d5620e...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=<try>
2d5620e 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

💔 Test failed - checks-travis

@RalfJung
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Trying commit 7d2bfd4 with merge a09bafe...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=<try>
a09bafe 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

💔 Test failed - checks-travis

@RalfJung
Copy link
Member Author

Found it! There was a bug in the alignment we used for the system allocator (malloc and friends).

I expanded the heap_allocator test to cover this on all platforms now.

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

📌 Commit 53c72ba has been approved by RalfJung

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Testing commit 53c72ba with merge 9dd1ff5...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=RalfJung
9dd1ff5 
use intptrcast for heap_allocator test; then it should work on Windows
@RalfJung
Copy link
Member Author

@bors r- retry

@RalfJung
Copy link
Member Author

Aaaand found a bug in libstd. ;) rust-lang/rust#62251

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

📌 Commit cb6d4f0 has been approved by RalfJung

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Testing commit cb6d4f0 with merge cc5878b...

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=RalfJung
cc5878b 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

💥 Test timed out

@RalfJung
Copy link
Member Author

@bors retry

bors added a commit that referenced this pull request Jun 30, 2019
@bors
Auto merge of #809 - RalfJung:intptrcast, r=RalfJung
2a4c4ad 
use intptrcast for heap_allocator test; then it should work on Windows
@bors
Copy link
Collaborator

bors commented Jun 30, 2019

⌛ Testing commit cb6d4f0 with merge 2a4c4ad...

@bors
Copy link
Collaborator

bors commented Jun 30, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 2a4c4ad to master...

@bors bors merged commit cb6d4f0 into rust-lang:master Jun 30, 2019
@bors bors mentioned this pull request Jun 30, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@RalfJung @bors