Re: [PATCH 1/3] arch-mips.inc: don't override TRANSLATED_TARGET_ARCH

[Mark Hatle <mark.hatle@windriver.com>]

On 6/18/15 8:13 AM, Dmitry Eremin-Solenikov wrote:
> Currently MIPS64 N32 is broken. There is internal disagreement
> between TARGET_ARCH (which doesn't contain ABIEXTENSION) and
> TRANSLATED_TARGET_ARCH (which contains ABIEXTENSION). ABI is already
> encoded into the TARGET_OS. ARM tunes in the same situation override
> neither the TARGET_ARCH nor the TRANSLATED_TARGET_ARCH. So let's drop
> this override.

This series won't work properly, unless I'm reading something incorrectly.

You won't be able to build/install a tri-lib system after this change, as
something needs to be there to differential between MIPS32 (o32), MIPS64 (n32)
and MIPS64 (n64).

Currently this is done via the ABIEXTENSION value.

What is currently broken w/ MIPS64 N32?  We put in a number of fixes for this
problem and SDK generation in the YP 1.8 time frame.  Perhaps something has
changed since then or maybe the fixes were not as complete as we thought?

--Mark

> Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> ---
>  meta/conf/machine/include/mips/arch-mips.inc | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/meta/conf/machine/include/mips/arch-mips.inc b/meta/conf/machine/include/mips/arch-mips.inc
> index c41fa5e..02626d3 100644
> --- a/meta/conf/machine/include/mips/arch-mips.inc
> +++ b/meta/conf/machine/include/mips/arch-mips.inc
> @@ -100,7 +100,3 @@ TUNE_FEATURES_tune-mips64el-nf = "n64"
>  BASE_LIB_tune-mips64el-nf = "lib64"
>  MIPSPKGSFX_VARIANT_tune-mips64el-nf = "${TUNE_ARCH}"
>  PACKAGE_EXTRA_ARCHS_tune-mips64el-nf = "mips64el-nf"
> -
> -# On mips we need to redefine this to include the ABIEXTENSION
> -# we can avoid the python bit as there are no _ or - to translate
> -TRANSLATED_TARGET_ARCH = "${TARGET_ARCH}${ABIEXTENSION}"
>