gallium/util: rewrite global constructor system for half floats (GCC/MSVC only!)
authorLuca Barbieri <luca@luca-barbieri.com>
Thu, 1 Apr 2010 16:40:27 +0000 (18:40 +0200)
committerLuca Barbieri <luca@luca-barbieri.com>
Thu, 1 Apr 2010 16:52:02 +0000 (18:52 +0200)
NOTE: this commit will cause Gallium to fail to build on any compiler
      except GCC, the Microsoft C compiler and compatible compilers that
      claim to be one of those.

This commit removes the u_gctors.cpp mechanism, in favor of using
compiler-specific syntax to add global constructors from C files.

This solves the problem of u_gctors.o not being pulled from static
libraries and avoids using C++.

However, it needs compiler-specific support for every compiler.

The Microsoft C compiler support has not been tested.

progs/gallium/unit/u_format_test.c
src/gallium/auxiliary/Makefile
src/gallium/auxiliary/util/u_gctors.cpp [deleted file]
src/gallium/auxiliary/util/u_half.c
src/gallium/auxiliary/util/u_half.h
src/gallium/auxiliary/util/u_init.h [new file with mode: 0644]

index 6ccef2361731c6febdd1cc959c0c6aeedffd1076..ca4c92f5b7ec42c4093ec0e1fb59834b17d5ef0b 100644 (file)
@@ -404,7 +404,6 @@ int main(int argc, char **argv)
 {
    boolean success;
 
-   util_half_init_tables();
    util_format_s3tc_init();
 
    success = test_all();
index 14c0fb1840a543bb2a00da2215a9e447fc76f69e..1db4aaa43987e4fbd36d7eb5f87ff4ab4b266bd9 100644 (file)
@@ -139,9 +139,6 @@ C_SOURCES = \
        #vl/vl_csc.c \
        #vl/vl_shader_build.c \
 
-CPP_SOURCES = \
-       util/u_gctors.cpp
-
 GALLIVM_SOURCES = \
         gallivm/lp_bld_alpha.c \
         gallivm/lp_bld_arit.c \
diff --git a/src/gallium/auxiliary/util/u_gctors.cpp b/src/gallium/auxiliary/util/u_gctors.cpp
deleted file mode 100644 (file)
index 3d2dc5b..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-/* this file uses the C++ global constructor mechanism to automatically
-   initialize global data
-
-   __attribute__((constructor)) allows to do this in C, but is GCC-only
-*/
-
-#include "u_half.h"
-
-struct util_gctor_t
-{
-       util_gctor_t()
-       {
-               util_half_init_tables();
-       }
-};
-
-static struct util_gctor_t util_gctor;
index 525ee7b1ecdf1540e75cc83f8c2b06bce4e3a7d1..4611b201b95ec5d44b50b408fbbb974a32f53c5a 100644 (file)
@@ -50,6 +50,7 @@
  */
 
 #include "util/u_half.h"
+#include "util/u_init.h"
 
 uint32_t util_half_to_float_mantissa_table[2048];
 uint32_t util_half_to_float_exponent_table[64];
@@ -57,7 +58,7 @@ uint32_t util_half_to_float_offset_table[64];
 uint16_t util_float_to_half_base_table[512];
 uint8_t util_float_to_half_shift_table[512];
 
-void util_half_init_tables(void)
+static void util_half_init_tables(void)
 {
        int i;
 
@@ -153,3 +154,5 @@ void util_half_init_tables(void)
                util_float_to_half_shift_table[256 + i] = util_float_to_half_shift_table[i];
        }
 }
+
+UTIL_INIT(util_half_init_tables);
index 0ca660faf3903fb5ba2c52bedbe7c5da06e6236f..5afdd925a624aa5bd121f44be868c327da1ebf37 100644 (file)
@@ -4,7 +4,6 @@
 #include "pipe/p_compiler.h"
 #include "u_math.h"
 
-
 #ifdef __cplusplus
 extern "C" {
 #endif
@@ -59,10 +58,6 @@ util_float_to_half(float f)
        return util_floatui_to_half(i.ui);
 }
 
-/* called by u_gctors.cpp, which defines the prototype itself */
-void util_half_init_tables(void);
-
-
 #ifdef __cplusplus
 }
 #endif
diff --git a/src/gallium/auxiliary/util/u_init.h b/src/gallium/auxiliary/util/u_init.h
new file mode 100644 (file)
index 0000000..edbcf2d
--- /dev/null
@@ -0,0 +1,51 @@
+/*
+ * Copyright 2010 Luca Barbieri
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining
+ * a copy of this software and associated documentation files (the
+ * "Software"), to deal in the Software without restriction, including
+ * without limitation the rights to use, copy, modify, merge, publish,
+ * distribute, sublicense, and/or sell copies of the Software, and to
+ * permit persons to whom the Software is furnished to do so, subject to
+ * the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the
+ * next paragraph) shall be included in all copies or substantial
+ * portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+ * IN NO EVENT SHALL THE COPYRIGHT OWNER(S) AND/OR ITS SUPPLIERS BE
+ * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+ * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+ * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ *
+ **************************************************************************/
+
+#ifndef U_INIT_H
+#define U_INIT_H
+
+/* Use UTIL_INIT(f) to have f called at program initialization.
+   Note that it is only guaranteed to be called if any symbol in the
+   .c file it is in sis referenced by the program.
+
+   UTIL_INIT functions are called in arbitrary order.
+*/
+
+#ifdef __cplusplus
+/* use a C++ global constructor */
+#define UTIL_INIT(f) struct f##__gctor_t {f##__gctor_t() {x();}} f##__gctor;
+#elif defined(_MSC_VER)
+/* add a pointer to the section where MSVC stores global constructor pointers */
+/* see http://blogs.msdn.com/vcblog/archive/2006/10/20/crt-initialization.aspx and
+   http://stackoverflow.com/questions/1113409/attribute-constructor-equivalent-in-vc */
+#define UTIL_INIT(f) static void __cdecl f##__init(void) {f();}; __declspec(allocate(".CRT$XCU")) void (__cdecl* f##__xcu)(void) = f##__init;
+#elif defined(__GNUC__)
+#define UTIL_INIT(f) static void f##__init(void) __attribute__((constructor)); static void f##__init(void) {f();}
+#else
+#error Unsupported compiler: please find out how to implement global initializers in C on it
+#endif
+
+#endif
+