summaryrefslogtreecommitdiffstats
path: root/package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch
diff options
context:
space:
mode:
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>2016-11-06 22:19:38 +0100
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>2016-11-06 22:21:07 +0100
commita1ca3dcb590852880770fe879ea59b6d8222b010 (patch)
tree200cb3318981568713fe850f647b68b0c76bf03e /package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch
parentd02655cd2157636d6150daf3a1344b25141780c1 (diff)
downloadbuildroot-a1ca3dcb590852880770fe879ea59b6d8222b010.tar.gz
buildroot-a1ca3dcb590852880770fe879ea59b6d8222b010.zip
jamvm: fix build with uClibc on i386/x86_64
Commit 0b6b67f34d6d4aeb340bbca579dbf85363c3f4ea ("jamvm: add patch to fix musl build") introduced a patch to fix the jamvm build with the musl C library. While the commit log pretends that the build was still working with uClibc, it is not correct: it no longer builds fine with uClibc on i386/x86_64, because the Buildroot default configuration for uClibc doesn't enable <fenv.h> support. Therefore this commit adapts the patch to use <fenv.h> if available (which is the case with musl), and otherwise fall back to <fpu_control.h>, which is available in uClibc. Thanks to Waldemar for the investigation. Fixes: http://autobuild.buildroot.net/results/325a50d15e1836b31df4e84ba83b296abfb73041/ Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Diffstat (limited to 'package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch')
-rw-r--r--package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch108
1 files changed, 108 insertions, 0 deletions
diff --git a/package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch b/package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch
new file mode 100644
index 0000000000..78ee9b7e18
--- /dev/null
+++ b/package/jamvm/0001-Use-fenv.h-when-available-instead-of-fpu_control.h.patch
@@ -0,0 +1,108 @@
+From ecd4eceae98cfb1c83133bdeaa9095546ca8b7c6 Mon Sep 17 00:00:00 2001
+From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
+Date: Thu, 26 May 2016 15:05:48 +0200
+Subject: [PATCH] Use <fenv.h> when available instead of <fpu_control.h>
+
+musl libc (http://musl-libc.org lack the non-standard <fpu_control.h>
+header, which is used in src/os/linux/{i386,x86_64}/init.c files to
+setup the floating point precision. This patch makes it use the
+standard C <fenv.h> header instead when available.
+
+Original patch at Felix Janda at
+https://sourceforge.net/p/jamvm/patches/6/, adapted to still use
+<fpu_control.h> if <fenv.h> is not provided.
+
+Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
+---
+ configure.ac | 2 +-
+ src/os/linux/i386/init.c | 15 +++++++++++++++
+ src/os/linux/x86_64/init.c | 15 +++++++++++++--
+ 3 files changed, 29 insertions(+), 3 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 19f77e6..ce59a3e 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -279,7 +279,7 @@ fi
+
+ dnl Checks for header files.
+ AC_HEADER_STDC
+-AC_CHECK_HEADERS(sys/time.h unistd.h endian.h sys/param.h locale.h alloca.h)
++AC_CHECK_HEADERS(sys/time.h unistd.h endian.h sys/param.h locale.h alloca.h fenv.h)
+
+ if test "$enable_zip" != no; then
+ AC_CHECK_HEADER(zlib.h,,AC_MSG_ERROR(zlib.h is missing))
+diff --git a/src/os/linux/i386/init.c b/src/os/linux/i386/init.c
+index d9c6648..8fefe7d 100644
+--- a/src/os/linux/i386/init.c
++++ b/src/os/linux/i386/init.c
+@@ -19,18 +19,33 @@
+ * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ */
+
++#include "config.h"
++
++#if defined(HAVE_FENV_H)
++#include <fenv.h>
++#else
+ #include <fpu_control.h>
++#endif
+
+ /* Change floating point precision to double (64-bit) from
+ * the extended (80-bit) Linux default. */
+
+ void setDoublePrecision() {
++#if defined(HAVE_FENV_H)
++ fenv_t fenv;
++
++ fegetenv(&fenv);
++ fenv.__control_word &= ~0x300; /* _FPU_EXTENDED */
++ fenv.__control_word |= 0x200; /* _FPU_DOUBLE */
++ fesetenv(&fenv);
++#else
+ fpu_control_t cw;
+
+ _FPU_GETCW(cw);
+ cw &= ~_FPU_EXTENDED;
+ cw |= _FPU_DOUBLE;
+ _FPU_SETCW(cw);
++#endif
+ }
+
+ void initialisePlatform() {
+diff --git a/src/os/linux/x86_64/init.c b/src/os/linux/x86_64/init.c
+index 9d55229..b42b14e 100644
+--- a/src/os/linux/x86_64/init.c
++++ b/src/os/linux/x86_64/init.c
+@@ -19,7 +19,11 @@
+ * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ */
+
+-#ifdef __linux__
++#include "config.h"
++
++#if defined(HAVE_FENV_H)
++#include <fenv.h>
++#else
+ #include <fpu_control.h>
+ #endif
+
+@@ -30,7 +34,14 @@
+ */
+
+ void setDoublePrecision() {
+-#ifdef __linux__
++#if defined(HAVE_FENV_H)
++ fenv_t fenv;
++
++ fegetenv(&fenv);
++ fenv.__control_word &= ~0x300; /*_FPU_EXTENDED */
++ fenv.__control_word |= 0x200; /*_FPU_DOUBLE */
++ fesetenv(&fenv);
++#else
+ fpu_control_t cw;
+
+ _FPU_GETCW(cw);
+--
+2.7.4
+
OpenPOWER on IntegriCloud