- changed status to open
- removed comment
Make signbit in Vectors work on more architectures
Issue #513
resolved
Several C++ compilers cannot handle std::signbit. There is also a namespace problem when using the same identifier Vectors_SGN for different precisions (real*4 and real*8). In addition, kifpos is implemented incorrectly on several architectures. The attached patch corrects these.
Keyword:
Comments (3)
-
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 - removed comment
- changed status to resolved
Applied.
- Log in to comment