ManevilleF / hexx

Hexagonal tools lib in rust
Apache License 2.0
276 stars 22 forks source link

Experimental SIMD support #137

Closed ManevilleF closed 6 months ago

ManevilleF commented 6 months ago

Addresses #59

Work done:

SIMD

Other

Thoughts

I added a benchmark that shows that the SIMD sum is slower than a regular sum in release mode. So either my Mac is not supported by wide simd operations and is falling back to regular slower sums, or I'm using the crate wrong.

Also the min/ max methods are not using SIMD at all, wide provides them but the implementation is not using simd optimizations.

I'm not sure that it's worth it.

alice-i-cecile commented 6 months ago

Yeah without benchmarks showing clear performance gains I definitely wouldn't add this.