public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] powerpc: Move around math-related Implies
@ 2018-05-11 19:36 Tulio Magno Quites Machado Filho
  2018-05-11 20:21 ` Gabriel F. T. Gomes
  0 siblings, 1 reply; 5+ messages in thread
From: Tulio Magno Quites Machado Filho @ 2018-05-11 19:36 UTC (permalink / raw)
  To: libc-alpha; +Cc: gabriel

Currently, powerpc, powerpc64, and powerpc64le imply the same set of
subdirectories from sysdeps/ieee754: flt-32, dbl-64, ldbl-128ibm, and
ldbl-opt.  In preparation for the transition of the long double format -
from IBM Extended Precision to IEEE 754 128-bits floating-point - on
powerpc64le, this patch splits the shared Implies file into three
separate files (one for each of the powerpc architectures), without
changing their contents.  Future patches will modify powerpc64le.

2018-05-11  Tulio Magno Quites Machado Filho  <tuliom@linux.ibm.com>
	    Gabriel F. T. Gomes  <gabriel@inconstante.eti.br>

	* sysdeps/powerpc/Implies: Removed.  Previous contents copied to...
	* sysdeps/powerpc/powerpc32/Implies-after: ... here.
	* sysdeps/powerpc/powerpc64/be/Implies-after: ... here.
	* sysdeps/powerpc/powerpc64/le/Implies-before: ... and here.
---
 sysdeps/powerpc/{Implies => powerpc32/Implies-after} | 0
 sysdeps/powerpc/powerpc64/be/Implies-after           | 5 +++++
 sysdeps/powerpc/powerpc64/le/Implies-before          | 5 +++++
 3 files changed, 10 insertions(+)
 rename sysdeps/powerpc/{Implies => powerpc32/Implies-after} (100%)
 create mode 100644 sysdeps/powerpc/powerpc64/be/Implies-after

diff --git a/sysdeps/powerpc/Implies b/sysdeps/powerpc/powerpc32/Implies-after
similarity index 100%
rename from sysdeps/powerpc/Implies
rename to sysdeps/powerpc/powerpc32/Implies-after
diff --git a/sysdeps/powerpc/powerpc64/be/Implies-after b/sysdeps/powerpc/powerpc64/be/Implies-after
new file mode 100644
index 0000000000..78dba9510c
--- /dev/null
+++ b/sysdeps/powerpc/powerpc64/be/Implies-after
@@ -0,0 +1,5 @@
+# On PowerPC we use the IBM extended long double format.
+ieee754/ldbl-128ibm
+ieee754/ldbl-opt
+ieee754/dbl-64
+ieee754/flt-32
diff --git a/sysdeps/powerpc/powerpc64/le/Implies-before b/sysdeps/powerpc/powerpc64/le/Implies-before
index 48065141a9..7c20db4e97 100644
--- a/sysdeps/powerpc/powerpc64/le/Implies-before
+++ b/sysdeps/powerpc/powerpc64/le/Implies-before
@@ -1 +1,6 @@
+# On PowerPC we use the IBM extended long double format.
+ieee754/ldbl-128ibm
+ieee754/ldbl-opt
+ieee754/dbl-64
+ieee754/flt-32
 ieee754/float128
-- 
2.14.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] powerpc: Move around math-related Implies
  2018-05-11 19:36 [PATCH] powerpc: Move around math-related Implies Tulio Magno Quites Machado Filho
@ 2018-05-11 20:21 ` Gabriel F. T. Gomes
  2018-05-11 22:02   ` Tulio Magno Quites Machado Filho
  0 siblings, 1 reply; 5+ messages in thread
From: Gabriel F. T. Gomes @ 2018-05-11 20:21 UTC (permalink / raw)
  To: Tulio Magno Quites Machado Filho; +Cc: libc-alpha

On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:

>Currently, powerpc, powerpc64, and powerpc64le imply the same set of
>subdirectories from sysdeps/ieee754: flt-32, dbl-64, ldbl-128ibm, and
>ldbl-opt.  In preparation for the transition of the long double format -
>from IBM Extended Precision to IEEE 754 128-bits floating-point - on
>powerpc64le, this patch splits the shared Implies file into three
>separate files (one for each of the powerpc architectures), without
>changing their contents.  Future patches will modify powerpc64le.
>
>2018-05-11  Tulio Magno Quites Machado Filho  <tuliom@linux.ibm.com>
>	    Gabriel F. T. Gomes  <gabriel@inconstante.eti.br>
>
>	* sysdeps/powerpc/Implies: Removed.  Previous contents copied to...
>	* sysdeps/powerpc/powerpc32/Implies-after: ... here.
>	* sysdeps/powerpc/powerpc64/be/Implies-after: ... here.
>	* sysdeps/powerpc/powerpc64/le/Implies-before: ... and here.

Do we still need to use the "-after" and "-before" mechanism after this
change?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] powerpc: Move around math-related Implies
  2018-05-11 20:21 ` Gabriel F. T. Gomes
@ 2018-05-11 22:02   ` Tulio Magno Quites Machado Filho
  2018-05-24 14:02     ` Gabriel F. T. Gomes
  0 siblings, 1 reply; 5+ messages in thread
From: Tulio Magno Quites Machado Filho @ 2018-05-11 22:02 UTC (permalink / raw)
  To: Gabriel F. T. Gomes; +Cc: libc-alpha

"Gabriel F. T. Gomes" <gabriel@inconstante.eti.br> writes:

> On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:
>
>>Currently, powerpc, powerpc64, and powerpc64le imply the same set of
>>subdirectories from sysdeps/ieee754: flt-32, dbl-64, ldbl-128ibm, and
>>ldbl-opt.  In preparation for the transition of the long double format -
>>from IBM Extended Precision to IEEE 754 128-bits floating-point - on
>>powerpc64le, this patch splits the shared Implies file into three
>>separate files (one for each of the powerpc architectures), without
>>changing their contents.  Future patches will modify powerpc64le.
>>
>>2018-05-11  Tulio Magno Quites Machado Filho  <tuliom@linux.ibm.com>
>>	    Gabriel F. T. Gomes  <gabriel@inconstante.eti.br>
>>
>>	* sysdeps/powerpc/Implies: Removed.  Previous contents copied to...
>>	* sysdeps/powerpc/powerpc32/Implies-after: ... here.
>>	* sysdeps/powerpc/powerpc64/be/Implies-after: ... here.
>>	* sysdeps/powerpc/powerpc64/le/Implies-before: ... and here.
>
> Do we still need to use the "-after" and "-before" mechanism after this
> change?

Yes.  Without them the sysdeps order would be different.

-- 
Tulio Magno

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] powerpc: Move around math-related Implies
  2018-05-11 22:02   ` Tulio Magno Quites Machado Filho
@ 2018-05-24 14:02     ` Gabriel F. T. Gomes
  2018-05-25  1:51       ` Tulio Magno Quites Machado Filho
  0 siblings, 1 reply; 5+ messages in thread
From: Gabriel F. T. Gomes @ 2018-05-24 14:02 UTC (permalink / raw)
  To: Tulio Magno Quites Machado Filho; +Cc: libc-alpha

On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:

>"Gabriel F. T. Gomes" <gabriel@inconstante.eti.br> writes:
>
>> On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:
>>  
>>>	* sysdeps/powerpc/Implies: Removed.  Previous contents copied to...
>>>	* sysdeps/powerpc/powerpc32/Implies-after: ... here.
>>>	* sysdeps/powerpc/powerpc64/be/Implies-after: ... here.
>>>	* sysdeps/powerpc/powerpc64/le/Implies-before: ... and here.  
>>
>> Do we still need to use the "-after" and "-before" mechanism after this
>> change?  
>
>Yes.  Without them the sysdeps order would be different.

OK, then.  The patch looks good to me.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] powerpc: Move around math-related Implies
  2018-05-24 14:02     ` Gabriel F. T. Gomes
@ 2018-05-25  1:51       ` Tulio Magno Quites Machado Filho
  0 siblings, 0 replies; 5+ messages in thread
From: Tulio Magno Quites Machado Filho @ 2018-05-25  1:51 UTC (permalink / raw)
  To: libc-alpha

"Gabriel F. T. Gomes" <gabriel@inconstante.eti.br> writes:

> On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:
>
>>"Gabriel F. T. Gomes" <gabriel@inconstante.eti.br> writes:
>>
>>> On Fri, 11 May 2018, Tulio Magno Quites Machado Filho wrote:
>>>  
>>>>	* sysdeps/powerpc/Implies: Removed.  Previous contents copied to...
>>>>	* sysdeps/powerpc/powerpc32/Implies-after: ... here.
>>>>	* sysdeps/powerpc/powerpc64/be/Implies-after: ... here.
>>>>	* sysdeps/powerpc/powerpc64/le/Implies-before: ... and here.  
>>>
>>> Do we still need to use the "-after" and "-before" mechanism after this
>>> change?  
>>
>>Yes.  Without them the sysdeps order would be different.
>
> OK, then.  The patch looks good to me.

Committed as c1dc1e1b3487.

Thanks!

-- 
Tulio Magno

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-05-25  1:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-11 19:36 [PATCH] powerpc: Move around math-related Implies Tulio Magno Quites Machado Filho
2018-05-11 20:21 ` Gabriel F. T. Gomes
2018-05-11 22:02   ` Tulio Magno Quites Machado Filho
2018-05-24 14:02     ` Gabriel F. T. Gomes
2018-05-25  1:51       ` Tulio Magno Quites Machado Filho

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).