[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