Test “self_touching” fails on aarch64, ppc64le, s390x about earcut.hpp HOT 5 OPEN

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.

from earcut.hpp.

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.

from earcut.hpp.

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.

from earcut.hpp.

I had a similar problem in aarch64 which was solved by passing the flag -ffp-contract=off to the compiler.
Such flag disables MADD / MSUB instructions, which, as I found out, behave differently on aarch64 architectures when compared to x86.
Here is the topic that led me to do this: https://bugs.mysql.com/bug.php?id=82760

from earcut.hpp.

I had a similar problem in aarch64 which was solved by passing the flag -ffp-contract=off to the compiler. Such flag disables MADD / MSUB instructions, which, as I found out, behave differently on aarch64 architectures when compared to x86. Here is the topic that led me to do this: https://bugs.mysql.com/bug.php?id=82760

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.

from earcut.hpp.