-
-
Notifications
You must be signed in to change notification settings - Fork 357
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
Use wide
to speedup coverage map with stable toolchains
#3131
Comments
Attached benchmark from aarch64. |
If it's not 100% the same perf, we can also just use wide for stable and leave simd for nightly |
it's good |
This would also mean we could move to simd in stable if it ever lands, though |
his repo shows it's almost the same. so no reason to keep portable simd |
Well I think it's good to keep it around in order to switch back some day in the future |
Ppl are still working actively on the feature, and once it lands it'd be better to get rid of the extra dependency I feel. See |
I will gate different implementations via features so that we still keep that while allowing users to choose. |
I did a benchmark for SIMD implementation of coverage map: https://github.com/wtdcode/libafl_simd_bench
The motivation is that we can use
wide
to replacestd::simd
so that we no longer need nightly to gain the speedup. It looks like thatwide
has almost the same performance asstd::simd
. I can draft a PR if we agree to introducewide
.wdyt? @tokatoka @domenukk @rmalmain
The text was updated successfully, but these errors were encountered: