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
Test “self_touching” fails on aarch64, ppc64le, s390x #97
Comments
Seems like the allowed deviation need to be loosened a little for the failing fixtures to accommodate your architecture and compiler configuration. Functionally I wouldn't worry about it though. |
I agree, and I’m planning to patch in a looser bound downstream as I package this library for Fedora Linux. If it matters, the compiler is gcc 11.2.1, and there are no particularly exotic compiler flags in use. Is there any interest in special-casing this test to accommodate these architectures upstream? If not, it’s probably best to close this issue. Thanks for your feedback. |
I'm not averse to merging changes to support the library/tests on more platforms. That being said, unless we create travis-ci instances testing the builds on these architectures, there is no guarantee there won't be changes in the future breaking it for those arch's again. |
I had a similar problem in aarch64 which was solved by passing the flag |
Thanks. This is a good suggestion. I can confirm that it fixes this test on all three architectures without needing to loosen the tolerance. Whether it is actually necessary or appropriate to disable floating-point contraction, or whether perhaps the tolerance is simply tighter than it needs to be, seems still to be an open question. For now, I’m going to leave the compiler flags as they are in Fedora, and keep loosening the tolerance. |
On
aarch64
,ppc64le
, ors390x
:On
x86_64
,i686
, orarmv7hl
/armhfp
:As you can see, the test output is not very informative. Changing the appropriate
setprecision(6)
tosetprecision(16)
intest/test.cpp
gives, on all three failing architectures:The text was updated successfully, but these errors were encountered: