Skip to content

Conversation

@gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Jul 3, 2019

The latest ctest version enabled the ABI roundtrip test by default, in which we initialize all types in Rust by default to some random bit-pattern, pass them to C, verify, modify, pass back to Rust, and verify.

This catches issues in the call ABI / calling convention.

This PR will silence those here for now.

@rust-highfive
Copy link

@gnzlbg: no appropriate reviewer found, use r? to override

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 5, 2019

@sunfishcode @alexcrichton wasi is failing to build because the memoffset dependency couldn't be resolved, could you point me to commit hashes that are known to build ? I'll include the fix here.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 5, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 5, 2019

📌 Commit 9f92863 has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Jul 5, 2019

⌛ Testing commit 9f92863 with merge 364cc8c...

bors added a commit that referenced this pull request Jul 5, 2019
Update ctest version

The latest ctest version enabled the ABI roundtrip test by default, in which we initialize all types in Rust by default to some random bit-pattern, pass them to C, verify, modify, pass back to Rust, and verify.

This catches issues in the call ABI / calling convention.

This PR will silence those here for now.
@bors
Copy link
Contributor

bors commented Jul 5, 2019

💔 Test failed - checks-travis

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 5, 2019 via email

@bors
Copy link
Contributor

bors commented Jul 5, 2019

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Jul 5, 2019

📌 Commit 9f92863 has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Jul 5, 2019

⌛ Testing commit 9f92863 with merge 5b2a64f...

bors added a commit that referenced this pull request Jul 5, 2019
Update ctest version

The latest ctest version enabled the ABI roundtrip test by default, in which we initialize all types in Rust by default to some random bit-pattern, pass them to C, verify, modify, pass back to Rust, and verify.

This catches issues in the call ABI / calling convention.

This PR will silence those here for now.
@bors
Copy link
Contributor

bors commented Jul 5, 2019

💔 Test failed - checks-travis

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 5, 2019

@bors: retry

@bors
Copy link
Contributor

bors commented Jul 5, 2019

⌛ Testing commit 9f92863 with merge 299a73f...

bors added a commit that referenced this pull request Jul 5, 2019
Update ctest version

The latest ctest version enabled the ABI roundtrip test by default, in which we initialize all types in Rust by default to some random bit-pattern, pass them to C, verify, modify, pass back to Rust, and verify.

This catches issues in the call ABI / calling convention.

This PR will silence those here for now.
@bors
Copy link
Contributor

bors commented Jul 5, 2019

☀️ Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing 299a73f to master...

@bors bors merged commit 9f92863 into rust-lang:master Jul 5, 2019
@bors bors mentioned this pull request Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants