-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Update stdsimd #58675
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
Update stdsimd #58675
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
@bors: r+ p=1 higher priority b/c this is likely to bounce a few times and it'd be good to weed that out |
📌 Commit 5490797732b3596908b1778e98fcb2e8ec0f483b has been approved by |
So I tried it locally, but not for arm / aarch64 / etc. and the recent ACLE changes might still need some ironing. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
So I can't reproduce this locally, I get a SIGSEV trying to build the docs. @bors: r- |
@gnzlbg: 🔑 Insufficient privileges: Not in reviewers |
@gnzlbg you can try with Docker image that failed on Travis. # depending on system configuration it may require `sudo`
./src/ci/docker/run.sh x86_64-gnu-llvm-6.0 |
Do they run on macos? (I have docker installed) |
I don't have Mac and I don't know how Docker works there (it allows running Linux via VM?). |
@QuietMisdreavus i've pushed a fix that might cause the aarch64 to contain docs of arm ACLE intrinsics and vice-versa, i don't know the best way to fix that. Basically, the arm and aarch64 docs include the ACLE docs, where ACLE has some intrinsics that are only for arm, and some for aarch64, and some for both. When compiling all targets with One could manually import everything from acle manually into the arm and aarch64 submodules, but ACLE is such a mess that this is, at this moment, not really feasible. Maybe when ACLE is fully implemented, we can do this once. cc @alexcrichton @Centril this should be ready2go |
@bors: r+ |
📌 Commit 2cf6e91 has been approved by |
Unblocks other work on edition migration; thus: @bors p=5 |
Update stdsimd This updates stdsimd to a Rust2015 / Rust2018 compatible version. Once this is merged it should be possible to migrate libcore and libstd to Rust2018. Once that happens, we can just require the 2018 edition in stdsimd.
💔 Test failed - checks-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@Centril something failed, I don't know what or why, but it appears spurious. |
Looks like the embedded book and the RLS failed to pass tests? Not totally sure the connection, but the test output logs are farther up the log. |
The raw logs are a pain to read:
And there are 305 occurences of error in the raw log. I went through most of them (so many its hard to tell if I missed any or not), and haven't found anything related to Looking for Does anybody have an idea about what's going on? Maybe this is just a spurious failure. |
embedded book fails all over the place like here:
I don't want to post RLS error here because it's huge and spurious. Neither or them should be affected by this PR. |
@Centril @alexcrichton can we give this another try ? |
@bors: r+ delegate+ |
✌️ @gnzlbg can now approve this pull request |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 2cf6e91 has been approved by |
Update stdsimd This updates stdsimd to a Rust2015 / Rust2018 compatible version. Once this is merged it should be possible to migrate libcore and libstd to Rust2018. Once that happens, we can just require the 2018 edition in stdsimd.
☀️ Test successful - checks-travis, status-appveyor |
This updates stdsimd to a Rust2015 / Rust2018 compatible version. Once this is merged it should be possible to migrate libcore and libstd to Rust2018. Once that happens, we can just require the 2018 edition in stdsimd.