[6/7] ldbl-128ibm: Let long double files have specific compiler flags

Message ID 20200214161051.32579-7-murphyp@linux.vnet.ibm.com
State New
Headers show
Series
  • Remaining ldbl-128ibm-compat support
Related show

Commit Message

Paul E. Murphy Feb. 14, 2020, 4:10 p.m.
From: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>


Soon, powerpc64le will need to provide extra compiler flags to the long
double files in order to continue to build using the IBM 128-bit
extended floating point type as long double.
---
 sysdeps/ieee754/ldbl-128ibm/Makefile | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

-- 
2.21.1

Comments

Paul E Murphy March 3, 2020, 8:18 p.m. | #1
Ping.  Will commit friday if no requests for improvement.

On 2/14/20 10:10 AM, Paul E. Murphy wrote:
> From: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>

> 

> Soon, powerpc64le will need to provide extra compiler flags to the long

> double files in order to continue to build using the IBM 128-bit

> extended floating point type as long double.

> ---

>   sysdeps/ieee754/ldbl-128ibm/Makefile | 22 +++++++++++++++++++---

>   1 file changed, 19 insertions(+), 3 deletions(-)

> 

> diff --git a/sysdeps/ieee754/ldbl-128ibm/Makefile b/sysdeps/ieee754/ldbl-128ibm/Makefile

> index bdba6cc6b5..c38d4f6ac2 100644

> --- a/sysdeps/ieee754/ldbl-128ibm/Makefile

> +++ b/sysdeps/ieee754/ldbl-128ibm/Makefile

> @@ -9,8 +9,24 @@ tests += tst-strtold-ldbl-128ibm

>   $(objpfx)tst-strtold-ldbl-128ibm: $(libm)

>   endif

> 

> +ldbl-tests = test-fmodl-ldbl-128ibm test-remainderl-ldbl-128ibm \

> +	     test-remquol-ldbl-128ibm test-canonical-ldbl-128ibm \

> +	     test-totalorderl-ldbl-128ibm

> +

>   ifeq ($(subdir),math)

> -tests += test-fmodl-ldbl-128ibm test-remainderl-ldbl-128ibm \

> -	 test-remquol-ldbl-128ibm test-canonical-ldbl-128ibm \

> -	 test-totalorderl-ldbl-128ibm

> +tests += $(ldbl-tests)

>   endif

> +

> +# Long double files may need extra CFLAGS.

> +ldbl-128ibm-routines = s_nexttoward s_nexttowardf \

> +		       $(type-ldouble-routines) \

> +		       $(subst F,$(type-ldouble-suffix),$(libm-compat-calls)) \

> +		       $(subst F,$(type-ldouble-suffix),$(libm-calls)) \

> +		       $(subst F,$(type-ldouble-suffix),$(calls)) \

> +		       $(foreach f,$(libm-narrow-fns), \

> +				 $(subst F,$(f), \

> +					 $(libm-narrow-types-ldouble-yes)))

> +$(foreach suf,$(all-object-suffixes), \

> +	  $(foreach r,$(ldbl-128ibm-routines) $(ldbl-tests), \

> +		    $(objpfx)$(r)$(suf))): \

> +  CFLAGS += $(type-ldouble-CFLAGS)

>
Paul E Murphy March 6, 2020, 7:38 p.m. | #2
And committed.

On 3/3/20 2:18 PM, Paul E Murphy wrote:
> Ping.  Will commit friday if no requests for improvement.

> 

> On 2/14/20 10:10 AM, Paul E. Murphy wrote:

>> From: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>

>>

>> Soon, powerpc64le will need to provide extra compiler flags to the long

>> double files in order to continue to build using the IBM 128-bit

>> extended floating point type as long double.

>> ---

>>   sysdeps/ieee754/ldbl-128ibm/Makefile | 22 +++++++++++++++++++---

>>   1 file changed, 19 insertions(+), 3 deletions(-)

Patch

diff --git a/sysdeps/ieee754/ldbl-128ibm/Makefile b/sysdeps/ieee754/ldbl-128ibm/Makefile
index bdba6cc6b5..c38d4f6ac2 100644
--- a/sysdeps/ieee754/ldbl-128ibm/Makefile
+++ b/sysdeps/ieee754/ldbl-128ibm/Makefile
@@ -9,8 +9,24 @@  tests += tst-strtold-ldbl-128ibm
 $(objpfx)tst-strtold-ldbl-128ibm: $(libm)
 endif
 
+ldbl-tests = test-fmodl-ldbl-128ibm test-remainderl-ldbl-128ibm \
+	     test-remquol-ldbl-128ibm test-canonical-ldbl-128ibm \
+	     test-totalorderl-ldbl-128ibm
+
 ifeq ($(subdir),math)
-tests += test-fmodl-ldbl-128ibm test-remainderl-ldbl-128ibm \
-	 test-remquol-ldbl-128ibm test-canonical-ldbl-128ibm \
-	 test-totalorderl-ldbl-128ibm
+tests += $(ldbl-tests)
 endif
+
+# Long double files may need extra CFLAGS.
+ldbl-128ibm-routines = s_nexttoward s_nexttowardf \
+		       $(type-ldouble-routines) \
+		       $(subst F,$(type-ldouble-suffix),$(libm-compat-calls)) \
+		       $(subst F,$(type-ldouble-suffix),$(libm-calls)) \
+		       $(subst F,$(type-ldouble-suffix),$(calls)) \
+		       $(foreach f,$(libm-narrow-fns), \
+				 $(subst F,$(f), \
+					 $(libm-narrow-types-ldouble-yes)))
+$(foreach suf,$(all-object-suffixes), \
+	  $(foreach r,$(ldbl-128ibm-routines) $(ldbl-tests), \
+		    $(objpfx)$(r)$(suf))): \
+  CFLAGS += $(type-ldouble-CFLAGS)