Message ID | 20180511193538.4445-1-tuliom@linux.ibm.com |
---|---|
State | New |
Headers | show |
Series | powerpc: Move around math-related Implies | expand |
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?
"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.
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.
"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!
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