Uh oh!
There was an error while loading. Please reload this page.
Update stdsimd - #58675
Conversation
rust-highfive
commented
Feb 23, 2019
(rust_highfive has picked a reviewer for you, use r? to override) |
rust-highfive
commented
Feb 23, 2019
|
rust-highfive
commented
Feb 23, 2019
(rust_highfive has picked a reviewer for you, use r? to override) |
rust-highfive
commented
Feb 23, 2019
|
alexcrichton
commented
Feb 23, 2019
@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 |
bors
commented
Feb 23, 2019
📌 Commit 5490797732b3596908b1778e98fcb2e8ec0f483b has been approved by |
gnzlbg
commented
Feb 23, 2019
So I tried it locally, but not for arm / aarch64 / etc. and the recent ACLE changes might still need some ironing. |
rust-highfive
commented
Feb 23, 2019
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 |
gnzlbg
commented
Feb 23, 2019
So I can't reproduce this locally, I get a SIGSEV trying to build the docs. @bors: r- |
bors
commented
Feb 23, 2019
@gnzlbg: 🔑 Insufficient privileges: Not in reviewers |
mati865
commented
Feb 23, 2019
@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) |
mati865
commented
Feb 24, 2019
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 |
alexcrichton
commented
Feb 25, 2019
@bors: r+ |
bors
commented
Feb 25, 2019
📌 Commit 2cf6e91 has been approved by |
Centril
commented
Feb 25, 2019
Unblocks other work on edition migration; thus: @bors p=5 |
bors
commented
Feb 25, 2019
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.
bors
commented
Feb 25, 2019
💔 Test failed - checks-travis |
rust-highfive
commented
Feb 25, 2019
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 |
gnzlbg
commented
Feb 26, 2019
@Centril something failed, I don't know what or why, but it appears spurious. |
QuietMisdreavus
commented
Feb 26, 2019
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. |
gnzlbg
commented
Feb 26, 2019
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. |
mati865
commented
Feb 26, 2019
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. |
gnzlbg
commented
Feb 26, 2019
@Centril@alexcrichton can we give this another try ? |
alexcrichton
commented
Feb 26, 2019
@bors: r+ delegate+ |
bors
commented
Feb 26, 2019
✌️ @gnzlbg can now approve this pull request |
bors
commented
Feb 26, 2019
💡 This pull request was already approved, no need to approve it again.
|
bors
commented
Feb 26, 2019
📌 Commit 2cf6e91 has been approved by |
bors
commented
Feb 26, 2019
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.
bors
commented
Feb 26, 2019
☀️ 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.