From 0d9b87c83ec0a36a8ab251b6167ea323aadb91d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Wed, 3 Jul 2024 18:31:25 +0200 Subject: [PATCH] Math: add an XFAIL for packing -1.0l to 64-bit integers on ARM Mac. Worked on x86, works on ARM Linux (with GCC) as well. Not sure if it has anything to do with long doubles being 64bit there, but I don't think it should. --- src/Magnum/Math/Test/PackingTest.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/Magnum/Math/Test/PackingTest.cpp b/src/Magnum/Math/Test/PackingTest.cpp index cb340d94a..c26ff9f70 100644 --- a/src/Magnum/Math/Test/PackingTest.cpp +++ b/src/Magnum/Math/Test/PackingTest.cpp @@ -215,6 +215,9 @@ void PackingTest::packSigned() { { #if defined(CORRADE_TARGET_MSVC) || (defined(CORRADE_TARGET_ANDROID) && defined(CORRADE_TARGET_32BIT) && defined(CORRADE_TARGET_X86)) CORRADE_EXPECT_FAIL("Long double (de)normalization is broken on MSVC and 32-bit x86 Android."); + #elif defined(CORRADE_TARGET_APPLE) && defined(CORRADE_TARGET_ARM) + /* Returns std::numeric_limits::min() */ + CORRADE_EXPECT_FAIL("Negative signed long double (de)normalization is broken on ARM Mac."); #endif CORRADE_COMPARE(Math::pack(-1.0l), std::numeric_limits::min()+1); }