- changed status to open
- removed comment
Make signbit in Vectors work on more architectures
Issue #513
closed
Several C++ compilers cannot handle std::signbit. There is also a namespace problem when using the same identifier Vectors_SGN for different precisions (real4 and real8). In addition, kifpos is implemented incorrectly on several architectures. The attached patch corrects these.
Keyword:
Comments (4)
-
reporter -
- removed comment
The changes look good to me. I've tested the SSE and default versions and they all compile and pass the Vectors testsuite on Damiana, Datura and my laptop. They also compile on Scientific Linux 6 which previously caused problems for me.
I didn't test the Altivec, DoubleHummer or VSX changes but they look correct. Any errors should be caught by the testsuites, so assuming those pass, I think this should be be committed.
-
reporter - changed status to resolved
- removed comment
Applied.
-
- edited description
- changed status to closed
- Log in to comment