This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH], PowerPC long double transition patches, v2, Patch #1 (disable long double multilib)


On Thu, Jun 21, 2018 at 12:58:06PM -0400, Michael Meissner wrote:
> On Wed, Jun 20, 2018 at 07:31:31PM -0500, Segher Boessenkool wrote:
> > On Wed, Jun 20, 2018 at 10:25:36AM -0400, Michael Meissner wrote:
> > > This code disables the automatic multilib creation unless you use the
> > > --with-advance-toolchain=<xxx> option and the Advance Toolchain directoy has
> > > been modified to have the lib64/ieee128 and/or lib64/ibm128 directories for
> > > multilib support.  This allows the multilib to still be created, but it is not
> > > enabled by default.
> > > 
> > > Alternatively, I have a patch that disables the IEEE/IBM long double multilib
> > > support completely.
> > 
> > So what are the advantages and the disadvantages of both approaches?
> > And, why do you think this one is preferable?
> 
> The main advantge is that it makes it a little easier for me to test things,
> since I only have to build one compiler instead of two.  But that is fairly
> minor.
> 
> Here is the alternate patch to eliminate the multilib support for IEEE/IBM long
> double.
> 
> [gcc]
> 2018-06-21  Michael Meissner  <meissner@linux.ibm.com>
> 
> 	* config.gcc (powerpc64le*): Remove multilib support for IEEE and
> 	IBM long double.
> 	* config/rs6000/rs6000.c (rs6000_option_override_internal):
> 	Likewise.
> 	* config/rs6000/rs6000.h (TARGET_IEEEQUAD_MULTILIB): Likewise.
> 	* config/rs6000/t-ldouble-linux64le-ibm: Delete, IEEE/IBM long
> 	double multilib no longer supported.
> 	* config/rs6000/t-ldouble-linux64le-ieee: Likewise.
> 	* doc/install.texi (PowerPC options): Delete information about
> 	IEEE/IBM long double multilibs.

This reverts r256775 (the commit message should say), except the linux64.h
parts, and it seems you just missed those?


Segher


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]