On 2025-01-28 21:01, Siddhesh Poyarekar wrote:
However, the previous test XFAILed the test for all PowerPC compiles
and your new one only XFAILs it for IBM128. That allows our two other
long double options to not be XFAILed. IEEE128 which is actually
expected
to PASS and --without-long-double-128/-mlong-double-64, which will now
show up as a FAIL.
So yeah, ppc_default_long_double_ieee isn't ideal because we'd need the
extra test, but ppc_default_long_double_ibm isn't enough either.
It's like we need a ppc_default_long_double_not_ieee or some such.
Maybe the following would work???
#if defined(__LONG_DOUBLE_IEEE128__)
#error "__LONG_DOUBLE_IEEE128__ is defined"
#endif
Ack, that makes sense. I've started a build with --without-long-
double-128 to test that configuration too and I'll post a v5 accordingly,
Sorry, hit Send too soon, do you think this would be better off being
called ppc_not_well_defined_denormals or something like that?
Thanks,
Sid
