Hi!

On Sun, Apr 26, 2020 at 11:35:21AM +0200, Iain Buclaw wrote:
> This patch adds power*-*-linux* as a supported target for libphobos.

Many thanks for doing this!

A problem though: libphobos/libdruntime is built for -m32 as well, but
that builds libphobos/libdruntime/config/powerpc64/callwithstack.S,
which cannot work (that file has all kinds of 64-bit only constructs and
instructions in it).

> diff --git a/libphobos/configure b/libphobos/configure
> index c2b49132fda..c923417532f 100755
> --- a/libphobos/configure
> +++ b/libphobos/configure
> @@ -13991,9 +13991,10 @@ fi
>                 ;;
>        mips*)   druntime_target_cpu_parsed="mips"
>                 ;;
> -      powerpc) druntime_target_cpu_parsed="powerpc"
> +      powerpc|powerpcle)
> +               druntime_target_cpu_parsed="powerpc"
>                 ;;
> -      powerpc64)
> +      powerpc64|powerpc64le)
>                 druntime_target_cpu_parsed="powerpc64"
>                 ;;
>        i[34567]86|x86_64)

We are a biarch target, so both powerpc-* and powerpc64-* configurations
can do both those configs (potentially, and the default for many configs).


Segher

Reply via email to