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  <meiss...@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

Reply via email to