Re: [Qemu-devel] [PATCH] configure: change endian cross compilation test

[Peter Maydell]

[added qemu-devel back again]

On 14 March 2012 17:51, Stuart Yoder <address@hidden> wrote:
> On Wed, Mar 14, 2012 at 12:17 PM, Peter Maydell
> <address@hidden> wrote:
>> Agreed. I think looking at the host CPUs we support the only ones
>> that aren't guaranteed either big or little endian are (a) mips
>> and (b) ARM. ARM already identifies big-endian by a compile-time
>> check, and it seems likely that MIPS could too.
>
> So, you're proposing just leaving the test as:
>
> # if cross compiling, cannot launch a program, so make a static guess
> case "$cpu" in
>  arm)
>    # ARM can be either way; ask the compiler which one we are
>    if check_define __ARMEB__; then
>      bigendian=yes
>    fi
>  ;;
>  hppa|m68k|mips|mips64|ppc|ppc64|s390|s390x|sparc|sparc64)
>    bigendian=yes
>  ;;
> esac

No, I'm proposing that you find out what the right check_define
for mips is and make mips|mips64 have a similar case to the arm
one. Google suggests the answer is __MIPSEB__; you can do a quick
check by firing up the mipsel/mips images from
http://people.debian.org/~aurel32/qemu/ under qemu :-)

-- PMM