-
Notifications
You must be signed in to change notification settings - Fork 99
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
Matrix Multiply SSE #65
Conversation
@StrangeZak I had to tweak the test compilation process so that the SSE |
Seems like everything i wanted to be done is done. @bvisness take a look and see if there's anything i should change. Also, i am worried about the _MM_TRANSPOSE4_PS i added since it is a helper macro. Is it safe to assume all everyone will have this macro? |
Also made the C++ tests actually run...😳
@StrangeZak I added C versions of the vector equality stuff you added. If the SSE tests pass in clang and gcc, and it works for you in Visual Studio, then I think it's safe to assume that the macro is widely supported. |
I think after this is merged and taken care of, I'm going to revisit #57 and rework our use of |
@bvisness Looks good to me. Shall i go ahead an do the merge ? |
Go for it! |
Done! |
Just making a PR so we have a common place for discussion.