tablegen: Don't use cbrtf in host tools
authorDerek Buitenhuis <derek.buitenhuis@gmail.com>
Tue, 15 Jan 2013 21:08:59 +0000 (16:08 -0500)
committerDerek Buitenhuis <derek.buitenhuis@gmail.com>
Tue, 15 Oct 2013 16:00:28 +0000 (17:00 +0100)
You cannot count on them being present on all systems, and you
cannot include libm.h in a host tool, so just hard code baseline
implementations.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
libavcodec/cbrt_tablegen.h
libavcodec/mpegaudio_tablegen.h

index 01963a3..60d900a 100644 (file)
@@ -36,12 +36,13 @@ static void cbrt_tableinit(void)
 {
     if (!cbrt_tab[(1<<13) - 1]) {
         int i;
+        /* cbrtf() isn't available on all systems, so we use powf(). */
         for (i = 0; i < 1<<13; i++) {
             union {
                 float f;
                 uint32_t i;
             } f;
-            f.f = cbrtf(i) * i;
+            f.f = powf(i, 1.0 / 3.0) * i;
             cbrt_tab[i] = f.i;
         }
     }
index a222f2c..a6c203f 100644 (file)
@@ -47,7 +47,8 @@ static void mpegaudio_tableinit(void)
         double value = i / 4;
         double f, fm;
         int e, m;
-        f  = value * cbrtf(value) * pow(2, (i & 3) * 0.25);
+        /* cbrtf() isn't available on all systems, so we use powf(). */
+        f  = value * powf(value, 1.0 / 3.0) * pow(2, (i & 3) * 0.25);
         fm = frexp(f, &e);
         m  = (uint32_t)(fm * (1LL << 31) + 0.5);
         e += FRAC_BITS - 31 + 5 - 100;
@@ -58,7 +59,8 @@ static void mpegaudio_tableinit(void)
     }
     for (exponent = 0; exponent < 512; exponent++) {
         for (value = 0; value < 16; value++) {
-            double f = (double)value * cbrtf(value) * pow(2, (exponent - 400) * 0.25 + FRAC_BITS + 5);
+            /* cbrtf() isn't available on all systems, so we use powf(). */
+            double f = (double)value * powf(value, 1.0 / 3.0) * pow(2, (exponent - 400) * 0.25 + FRAC_BITS + 5);
             expval_table_fixed[exponent][value] = llrint(f);
             expval_table_float[exponent][value] = f;
         }