[PATCH, r3], Add optional IEEE/IBM long double multilib support

Segher Boessenkool segher@kernel.crashing.org
Fri Jan 12 17:55:00 GMT 2018


On Fri, Jan 12, 2018 at 07:23:37AM -0500, Michael Meissner wrote:
> Of course it would be helpful, if I included the patch:

Yup :-)

> 2018-01-12  Michael Meissner  <meissner@linux.vnet.ibm.com>
> 
> 	* config.gcc (powerpc*-linux*-*): Add support for 64-bit little
> 	endian Linux systems to optionally enable multilibs for selecting
> 	the long double type if the user configured an explicit type.
> 	* config/rs6000/rs6000.h (TARGET_IEEEQUAD_MULTILIB): Indicate we
> 	have no long double multilibs if not defined.
> 	* config/rs6000/rs6000.c (rs6000_option_override_internal): Do not
> 	warn if the user used -mabi={ieee,ibm}longdouble and we built
> 	multilibs for long double.
> 	* config/rs6000/linux64.h (MULTILIB_DEFAULTS_IEEE): Define as the
> 	appropriate multilib option.
> 	(MULTILIB_DEFAULTS): Add MULTILIB_DEFAULTS_IEEE to the default
> 	multilib options.
> 	* config/rs6000/t-ldouble-linux64le-ibm: New configuration files
> 	for building long double multilibs.
> 	* config/rs6000/t-ldouble-linux64le-ieee: Likewise.


> +		case "${target}:${enable_multilib}:${with_long_double_format}" in
> +		    powerpc64le*:yes:ieee | powerpc64le*:yes:ibm)
> +			tm_defines="${tm_defines} TARGET_IEEEQUAD_MULTILIB=1"
> +			tmake_file="${tmake_file} rs6000/t-ldouble-linux64le-${with_long_double_format}"
> +			;;
> +		    *)
> +			:
> +			;;

You could just leave out those three lines default (or at least the :).

Looks fine I think.  Okay for trunk, thanks!


Segher



More information about the Gcc-patches mailing list