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

fix unrecognized platform-specific intrinsic function: simd_shuffle2 etc. #8

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Conversation

KonradHoeffner
Copy link
Contributor

@KonradHoeffner KonradHoeffner commented Aug 7, 2023

This fixes the errors: "unrecognized platform-specific intrinsic function: simd_shuffle2" etc.

Error reported in #7.
The error only occurred with the newest nightly versions

Update dependencies to their newest version.
Successfully tested with:

stable: 1.71.1 2023-08-03
nightly: 1.73.0 23-08-06

  • cargo +stable test
  • cargo +nightly test
  • cargo +nightly test --features packed_simd
  • cargo +stable afl build --release --test rsdict_fuzz --features fuzz
  • cargo +nightly afl build --release --test rsdict_fuzz --features fuzz

The last two report:

warning: unused import: `succinct::rank::RankSupport`
warning: unused import: `succinct::select::SelectSupport`

However I didn't remove them in this PR as this occurs with the previous version as well.

@KonradHoeffner KonradHoeffner changed the title Update dependencies fix unrecognized platform-specific intrinsic function: simd_shuffle2 etc. Aug 7, 2023
@sujayakar
Copy link
Owner

wonderful, thanks for the PR!

let me know if you'd like me to do a crates.io release too.

@sujayakar sujayakar merged commit 1733549 into sujayakar:master Aug 10, 2023
@KonradHoeffner
Copy link
Contributor Author

@sujayakar: You're welcome! A crates.io release would be great, as I have two crates that depend on rsdict.

@sujayakar
Copy link
Owner

@KonradHoeffner, done! 0.0.7 is on crates.io: https://crates.io/crates/rsdict

@KonradHoeffner
Copy link
Contributor Author

Fantastic, thanks!

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.

2 participants