-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Update stdsimd #58579
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 #58579
Conversation
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 |
cc @japaric looks like there are some issues while building stdsimd docs. Could you take a look? |
@gnzlbg it's easy to fix the build failure; what's going to be tricky is showing the right API under The problem is that the whole To show the right API with the "dox" feature we would have to add something like this to // arm/mod.rs
#[cfg(not(dox))] // <- ADD
pub use super::acle::*;
// ADD
#[cfg(dox)]
pub use super::acle::{dmb, dsb, /* plus dozens of other funcions */}; Basically re-export the right thing in each module, which is going to be a lot of work to do correctly and maintain. |
@japaric Would splitting the acle module in, say, |
I think the priority for now should be to unblock transitioning libstd to 2018 so that we can finish the transitions. |
Due to a new major feature landing on stdsimd this week (unfortunate timing), the library does not build in tree anymore (happens pretty much all the time anyways). What @taiki-e is doing is IMO the best way to proceed. Let's get stdsimd in Rust2015 mode building in tree again, which is not trivial (@taiki-e do you mind if I take over doing this?). Once that is merged, the Rust2018 PR for libstd can be updated, and we can just focus on solving Rust2018 related issues there. @taiki-e has already sent a PR fixing some of this, so I'm sure they will be able to nail all other issues in no time once we are there. |
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 Thanks! |
r? @gnzlbg