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.

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

Small improvements for the examples #62

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

verpeteren
Copy link

I started working on transitioning (https://github.com/verpeteren/rust-simd-noise/pull/50/files#top) from rust-simd-noise on top of the 64 bit changes (verpeteren/rust-simd-noise#51), but got kindof stuck because it seems that the Avx2, Sse2, Sse41 engines are not exported, only Scalar is available.

Upon further investigation, it seems that the examples were outdated and showed the same issue.

Please advice on how I can improve/continue with this.

cargo test

gives

   Compiling simdeez v2.0.0-dev3 (/data/Development/other/simdeez)
error[E0432]: unresolved import `simdeez::avx2`
 --> src/main.rs:3:14
  |
3 | use simdeez::avx2::*;
  |              ^^^^ could not find `avx2` in `simdeez`

error[E0432]: unresolved import `simdeez::sse2`
 --> src/main.rs:5:14
  |
5 | use simdeez::sse2::*;
  |              ^^^^ could not find `sse2` in `simdeez`

error[E0432]: unresolved import `simdeez::sse41`
 --> src/main.rs:6:14
  |
6 | use simdeez::sse41::*;
  |              ^^^^^ could not find `sse41` in `simdeez`

warning: unused import: `simdeez::scalar::*`
 --> src/main.rs:4:5
  |
4 | use simdeez::scalar::*;
  |     ^^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

For more information about this error, try `rustc --explain E0432`.
warning: `simdeez` (bin "simdeez" test) generated 1 warning
error: could not compile `simdeez` (bin "simdeez" test) due to 3 previous errors; 1 warning emitted
warning: build failed, waiting for other jobs to finish...

This PR is broken down into small atomic commits that should be easy to review

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.

1 participant