Open cdwfs opened 6 years ago
Wanna send us a pull request?
Sure, will do
@cdwfs do you see performance improvement while using SIMD on MSVC target? For me the Vector Benchmark is 50% slower using SIMD...
I see the same results: a ~50% slowdown when enabling SIMD. See the comments on my pull request #26.
Just did, thank you. Indeed x64 is irrelevant to the issue you mentioned, I was running my bench in x86. For me the way the lib was designed and coded make its hard to really benefit of SSE2 Instruction as there're still conversion, copy constructors, temp objects...
The checks in
mathfu/utilities.h
to enableMATHFU_COMPILE_WITH_SIMD
don't cover Visual Studio x64 targets. The code currently checks the existence (and value) of_M_IX86_FP
, which is only defined for x86 (32-bit) targets. SSE and SSE2 support is implicit on x64 targets, which can be handled with the following addition (or similar):