fix definition of CMPLX macros in complex.h to work in constant expressions
authorRich Felker <dalias@aerifal.cx>
Wed, 17 Dec 2014 21:34:41 +0000 (16:34 -0500)
committerRich Felker <dalias@aerifal.cx>
Wed, 17 Dec 2014 21:43:09 +0000 (16:43 -0500)
based on patches by Jens Gustedt. these macros need to be usable in
static initializers, and the old definitions were not.

there is no portable way to provide correct definitions for these
macros unless the compiler supports pure imaginary types. a portable
definition is provided for this case even though there are presently
no compilers that can use it. gcc and compatible compilers provide a
builtin function that can be used, but clang fails to support this and
instead requires a construct which is a constraint violation and which
is only a constant expression as a clang-specific extension.

since these macros are a namespace violation in pre-C11 profiles, and
since no known pre-C11 compilers provide any way to define them
correctly anyway, the definitions have been made conditional on C11.

include/complex.h

index e1af0d59a7bbee15ddcee7c9a364c1d29db04f81..23bab7d5593042e3c878707ad22dc86cdb843a1d 100644 (file)
@@ -114,12 +114,18 @@ long double creall(long double complex);
 #define cimagl(x) __CIMAG(x, long double)
 #endif
 
-#define __CMPLX(x, y, t) \
-       ((union { _Complex t __z; t __xy[2]; }){.__xy = {(x),(y)}}.__z)
-
+#if __STDC_VERSION__ >= 201112L
+#if defined(_Imaginary_I)
+#define __CMPLX(x, y, t) ((t)(x) + _Imaginary_I*(t)(y)))
+#elif defined(__clang__)
+#define __CMPLX(x, y, t) (+(_Complex t){ (t)(x), (t)(y) })
+#else
+#define __CMPLX(x, y, t) (__builtin_complex((t)(x), (t)(y)))
+#endif
 #define CMPLX(x, y) __CMPLX(x, y, double)
 #define CMPLXF(x, y) __CMPLX(x, y, float)
 #define CMPLXL(x, y) __CMPLX(x, y, long double)
+#endif
 
 #ifdef __cplusplus
 }