Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
On 05.07.2018 22:14, James Clarke wrote: Control: notfound -1 8.1.0-9 7.3.0-24thanks gcc-7/7.3.0-24 Control: found -1 7.3.0-24 Control: tags -1 upstream fixed-upstream Control: forwarded -1 https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00219.html On Thu, Jul 05, 2018 at 12:54:37PM +0200, Svante Signell wrote: On Thu, 2018-07-05 at 12:37 +0200, Svante Signell wrote: On Tue, 2018-07-03 at 05:42 +0200, Matthias Klose wrote: On 01.07.2018 00:02, Svante Signell wrote: reopen 897416 found 897416 7.3.0-24, 8.1.0-9 tags 897416 patch thanks Hi, the patch by Matthias was not enough to make gcc-7,8 to build properly. Three configure files were needed to be recreated from their corresponding configure.ac. Additionally, this patch use x86_64*- *- gnu* to also cover the future implementation of 64bit Hurd. this is not applied upstream. Please test the patch on trunk, forward it upstream and make sure it gets applied. It is already forwarded upstream (somewhat differently) by James Clarke: https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00218.html Additional info: gcc-7-7.3.2-24 and gcc-8-8.1.0-9 built fine on kfreebsd-amd64 with the patch in this bug report. Hi all, The patch I submitted upstream (v2) has been applied to trunk (r62452). I have modified it for the GCC Debian packaging (adding the src/ prefix to the paths) so it should Just Work; please could you use it for the next upload? I changed it to not patch the regenerated files, but instead to touch all the affected */configure.ac files using a comment. Matthias
Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
Control: notfound -1 8.1.0-9 7.3.0-24thanks gcc-7/7.3.0-24 Control: found -1 7.3.0-24 Control: tags -1 upstream fixed-upstream Control: forwarded -1 https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00219.html On Thu, Jul 05, 2018 at 12:54:37PM +0200, Svante Signell wrote: > On Thu, 2018-07-05 at 12:37 +0200, Svante Signell wrote: > > On Tue, 2018-07-03 at 05:42 +0200, Matthias Klose wrote: > > > On 01.07.2018 00:02, Svante Signell wrote: > > > > reopen 897416 > > > > found 897416 7.3.0-24, 8.1.0-9 > > > > tags 897416 patch > > > > thanks > > > > > > > > Hi, the patch by Matthias was not enough to make gcc-7,8 to build > > > > properly. Three configure files were needed to be recreated from > > > > their > > > > corresponding configure.ac. Additionally, this patch use x86_64*- > > > > *- > > > > gnu* > > > > to also cover the future implementation of 64bit Hurd. > > > > > > this is not applied upstream. Please test the patch on trunk, > > > forward > > > it upstream and make sure it gets applied. > > > > It is already forwarded upstream (somewhat differently) by James > > Clarke: https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00218.html > > Additional info: gcc-7-7.3.2-24 and gcc-8-8.1.0-9 built fine on > kfreebsd-amd64 with the patch in this bug report. Hi all, The patch I submitted upstream (v2) has been applied to trunk (r62452). I have modified it for the GCC Debian packaging (adding the src/ prefix to the paths) so it should Just Work; please could you use it for the next upload? Thanks, James >From 1c703f62283a33508fbd5d2a9b38dd7bfaeb3c83 Mon Sep 17 00:00:00 2001 From: James Clarke Date: Thu, 14 Jun 2018 15:04:47 +0100 Subject: [PATCH v2] Enable decimal float on x86_64 kFreeBSD and Hurd To: gcc-patc...@gcc.gnu.org config/ * dfp.m4 (enable_decimal_float): Enable for x86_64*-*-gnu* to catch x86_64 kFreeBSD and Hurd. gcc/ * configure: Regenerate. libdecnumber/ * configure: Regenerate. libgcc/ * configure: Regenerate. --- config/dfp.m4 | 2 +- gcc/configure | 2 +- libdecnumber/configure | 2 +- libgcc/configure | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/config/dfp.m4 b/src/config/dfp.m4 index 5b29089cec5..a137ddebf8c 100644 --- a/src/config/dfp.m4 +++ b/src/config/dfp.m4 @@ -21,7 +21,7 @@ Valid choices are 'yes', 'bid', 'dpd', and 'no'.]) ;; [ case $1 in powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \ -i?86*-*-elfiamcu | i?86*-*-gnu* | \ +i?86*-*-elfiamcu | i?86*-*-gnu* | x86_64*-*-gnu* | \ i?86*-*-mingw* | x86_64*-*-mingw* | \ i?86*-*-cygwin* | x86_64*-*-cygwin*) enable_decimal_float=yes diff --git a/src/gcc/configure b/src/gcc/configure index 60d373982fd..35564942bbf 100755 --- a/src/gcc/configure +++ b/src/gcc/configure @@ -7458,7 +7458,7 @@ else case $target in powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \ -i?86*-*-elfiamcu | i?86*-*-gnu* | \ +i?86*-*-elfiamcu | i?86*-*-gnu* | x86_64*-*-gnu* | \ i?86*-*-mingw* | x86_64*-*-mingw* | \ i?86*-*-cygwin* | x86_64*-*-cygwin*) enable_decimal_float=yes diff --git a/src/libdecnumber/configure b/src/libdecnumber/configure index 4cb732e80d4..b1588f4e884 100755 --- a/src/libdecnumber/configure +++ b/src/libdecnumber/configure @@ -4709,7 +4709,7 @@ else case $target in powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \ -i?86*-*-elfiamcu | i?86*-*-gnu* | \ +i?86*-*-elfiamcu | i?86*-*-gnu* | x86_64*-*-gnu* | \ i?86*-*-mingw* | x86_64*-*-mingw* | \ i?86*-*-cygwin* | x86_64*-*-cygwin*) enable_decimal_float=yes diff --git a/src/libgcc/configure b/src/libgcc/configure index b2f3f870844..f395474beac 100644 --- a/src/libgcc/configure +++ b/src/libgcc/configure @@ -4647,7 +4647,7 @@ else case $host in powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \ -i?86*-*-elfiamcu | i?86*-*-gnu* | \ +i?86*-*-elfiamcu | i?86*-*-gnu* | x86_64*-*-gnu* | \ i?86*-*-mingw* | x86_64*-*-mingw* | \ i?86*-*-cygwin* | x86_64*-*-cygwin*) enable_decimal_float=yes -- 2.17.0
Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
On Thu, 2018-07-05 at 12:37 +0200, Svante Signell wrote: > On Tue, 2018-07-03 at 05:42 +0200, Matthias Klose wrote: > > On 01.07.2018 00:02, Svante Signell wrote: > > > reopen 897416 > > > found 897416 7.3.0-24, 8.1.0-9 > > > tags 897416 patch > > > thanks > > > > > > Hi, the patch by Matthias was not enough to make gcc-7,8 to build > > > properly. Three configure files were needed to be recreated from > > > their > > > corresponding configure.ac. Additionally, this patch use x86_64*- > > > *- > > > gnu* > > > to also cover the future implementation of 64bit Hurd. > > > > this is not applied upstream. Please test the patch on trunk, > > forward > > it upstream and make sure it gets applied. > > It is already forwarded upstream (somewhat differently) by James > Clarke: https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00218.html Additional info: gcc-7-7.3.2-24 and gcc-8-8.1.0-9 built fine on kfreebsd-amd64 with the patch in this bug report.
Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
On Tue, 2018-07-03 at 05:42 +0200, Matthias Klose wrote: > On 01.07.2018 00:02, Svante Signell wrote: > > reopen 897416 > > found 897416 7.3.0-24, 8.1.0-9 > > tags 897416 patch > > thanks > > > > Hi, the patch by Matthias was not enough to make gcc-7,8 to build > > properly. Three configure files were needed to be recreated from > > their > > corresponding configure.ac. Additionally, this patch use x86_64*-*- > > gnu* > > to also cover the future implementation of 64bit Hurd. > > this is not applied upstream. Please test the patch on trunk, forward > it upstream and make sure it gets applied. It is already forwarded upstream (somewhat differently) by James Clarke: https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00218.html
Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
On 01.07.2018 00:02, Svante Signell wrote: reopen 897416 found 897416 7.3.0-24, 8.1.0-9 tags 897416 patch thanks Hi, the patch by Matthias was not enough to make gcc-7,8 to build properly. Three configure files were needed to be recreated from their corresponding configure.ac. Additionally, this patch use x86_64*-*-gnu* to also cover the future implementation of 64bit Hurd. this is not applied upstream. Please test the patch on trunk, forward it upstream and make sure it gets applied.
Bug#897416: gcc: Decimal float support is not enabled on kfreebsd-amd64
reopen 897416 found 897416 7.3.0-24, 8.1.0-9 tags 897416 patch thanks Hi, the patch by Matthias was not enough to make gcc-7,8 to build properly. Three configure files were needed to be recreated from their corresponding configure.ac. Additionally, this patch use x86_64*-*-gnu* to also cover the future implementation of 64bit Hurd. Thanks!# DP: Enable decimal float support on kfreebsd-amd64 --- a/src/gcc/configure.ac +++ b/src/gcc/configure.ac @@ -817,6 +817,7 @@ AC_ARG_ENABLE(__cxa_atexit, [], []) # Enable C extension for decimal float if target supports it. +# touch the file, adding decimal support for kfreebsd-amd64 in config/dfp.m4 GCC_AC_ENABLE_DECIMAL_FLOAT([$target]) dfp=`if test $enable_decimal_float != no; then echo 1; else echo 0; fi` --- a/src/libgcc/configure.ac +++ b/src/libgcc/configure.ac @@ -206,6 +206,7 @@ AC_CHECK_HEADERS(inttypes.h stdint.h std AC_HEADER_STDC # Check for decimal float support. +# touch the file, adding decimal support for kfreebsd-amd64 in config/dfp.m4 AC_CACHE_CHECK([whether decimal floating point is supported], [libgcc_cv_dfp], [AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ #include --- a/src/libdecnumber/configure.ac 2017-01-08 17:43:30.0 +0100 +++ b/src/libdecnumber/configure.ac 2018-06-30 18:41:50.117324000 +0200 @@ -77,6 +77,7 @@ # Default decimal format # If you change the defaults here, be sure to change them in the GCC directory also +# touch the file, adding decimal support for kfreebsd-amd64 in config/dfp.m4 AC_MSG_CHECKING([for decimal floating point]) GCC_AC_ENABLE_DECIMAL_FLOAT([$target]) --- a/src/config/dfp.m4 +++ b/src/config/dfp.m4 @@ -21,7 +21,7 @@ Valid choices are 'yes', 'bid', 'dpd', a [ case $1 in powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \ -i?86*-*-elfiamcu | i?86*-*-gnu* | \ +i?86*-*-elfiamcu | i?86*-*-gnu* | x86_64*-*-gnu* | \ i?86*-*-mingw* | x86_64*-*-mingw* | \ i?86*-*-cygwin* | x86_64*-*-cygwin*) enable_decimal_float=yes