Patchwork configure: detect endian via compile test

login
register
mail settings
Submitter Mike Frysinger
Date July 1, 2013, 3:30 a.m.
Message ID <1372649418-4987-1-git-send-email-vapier@gentoo.org>
Download mbox | patch
Permalink /patch/256001/
State New
Headers show

Comments

Mike Frysinger - July 1, 2013, 3:30 a.m.
This avoids needing to execute a program and keeping an (incomplete)
list when cross-compiling.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
 configure | 40 ++++++++++++++--------------------------
 1 file changed, 14 insertions(+), 26 deletions(-)
Richard Henderson - July 2, 2013, 3:19 p.m.
On 06/30/2013 08:30 PM, Mike Frysinger wrote:
> This avoids needing to execute a program and keeping an (incomplete)
> list when cross-compiling.
> 
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>

For the benefit of the list, this is a porting of the autoconf test.

Reviewed-by: Richard Henderson <rth@twiddle.net>


r~
James Hogan - Aug. 28, 2013, 8:21 a.m.
On 1 July 2013 04:30, Mike Frysinger <vapier@gentoo.org> wrote:
> This avoids needing to execute a program and keeping an (incomplete)
> list when cross-compiling.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>

This fixes mipsel cross compiling. I also checked it detected a mips
(be) compiler as big endian.

Tested-by: James Hogan <james.hogan@imgtec.com> [mips]

Can somebody please apply this. Maybe for stable too?

Cheers
James
Stefan Weil - Sept. 9, 2013, 7:30 p.m.
Am 28.08.2013 10:21, schrieb James Hogan:
> On 1 July 2013 04:30, Mike Frysinger <vapier@gentoo.org> wrote:
>> This avoids needing to execute a program and keeping an (incomplete)
>> list when cross-compiling. Signed-off-by: Mike Frysinger
>> <vapier@gentoo.org> 
> This fixes mipsel cross compiling. I also checked it detected a mips
> (be) compiler as big endian. Tested-by: James Hogan
> <james.hogan@imgtec.com> [mips] Can somebody please apply this. Maybe
> for stable too? Cheers James 

Ping? Aurelien, Anthony, who wants to commit this patch?
Richard already reviewed it.

See also http://patchwork.ozlabs.org/patch/268687/ for
another configure patch waiting for a commit.

Regards,
Stefan
Doug Goldstein - Sept. 26, 2013, 3:22 a.m.
On Mon, Sep 9, 2013 at 2:30 PM, Stefan Weil <stefan.weil@weilnetz.de> wrote:
> Am 28.08.2013 10:21, schrieb James Hogan:
>> On 1 July 2013 04:30, Mike Frysinger <vapier@gentoo.org> wrote:
>>> This avoids needing to execute a program and keeping an (incomplete)
>>> list when cross-compiling. Signed-off-by: Mike Frysinger
>>> <vapier@gentoo.org>
>> This fixes mipsel cross compiling. I also checked it detected a mips
>> (be) compiler as big endian. Tested-by: James Hogan
>> <james.hogan@imgtec.com> [mips] Can somebody please apply this. Maybe
>> for stable too? Cheers James
>
> Ping? Aurelien, Anthony, who wants to commit this patch?
> Richard already reviewed it.
>
> See also http://patchwork.ozlabs.org/patch/268687/ for
> another configure patch waiting for a commit.
>
> Regards,
> Stefan
>
>

Ping on getting this into master (and then over to stable).
Paolo Bonzini - Sept. 26, 2013, 8:54 p.m.
Il 26/09/2013 05:22, Doug Goldstein ha scritto:
> On Mon, Sep 9, 2013 at 2:30 PM, Stefan Weil <stefan.weil@weilnetz.de> wrote:
>> Am 28.08.2013 10:21, schrieb James Hogan:
>>> On 1 July 2013 04:30, Mike Frysinger <vapier@gentoo.org> wrote:
>>>> This avoids needing to execute a program and keeping an (incomplete)
>>>> list when cross-compiling. Signed-off-by: Mike Frysinger
>>>> <vapier@gentoo.org>
>>> This fixes mipsel cross compiling. I also checked it detected a mips
>>> (be) compiler as big endian. Tested-by: James Hogan
>>> <james.hogan@imgtec.com> [mips] Can somebody please apply this. Maybe
>>> for stable too? Cheers James
>>
>> Ping? Aurelien, Anthony, who wants to commit this patch?
>> Richard already reviewed it.
>>
>> See also http://patchwork.ozlabs.org/patch/268687/ for
>> another configure patch waiting for a commit.
>>
>> Regards,
>> Stefan
>>
>>
> 
> Ping on getting this into master (and then over to stable).
> 

Thanks Doug.  Anthony, Aurelien, can you commit it?

Paolo
Peter Maydell - Nov. 5, 2013, 5:36 p.m.
On 26 September 2013 21:54, Paolo Bonzini <pbonzini@redhat.com> wrote:
> Il 26/09/2013 05:22, Doug Goldstein ha scritto:
>> On Mon, Sep 9, 2013 at 2:30 PM, Stefan Weil <stefan.weil@weilnetz.de> wrote:
>>> Am 28.08.2013 10:21, schrieb James Hogan:
>>>> On 1 July 2013 04:30, Mike Frysinger <vapier@gentoo.org> wrote:
>>>>> This avoids needing to execute a program and keeping an (incomplete)
>>>>> list when cross-compiling. Signed-off-by: Mike Frysinger
>>>>> <vapier@gentoo.org>
>>>> This fixes mipsel cross compiling. I also checked it detected a mips
>>>> (be) compiler as big endian. Tested-by: James Hogan
>>>> <james.hogan@imgtec.com> [mips] Can somebody please apply this. Maybe
>>>> for stable too? Cheers James
>>>
>>> Ping? Aurelien, Anthony, who wants to commit this patch?
>>> Richard already reviewed it.
>>>
>>> See also http://patchwork.ozlabs.org/patch/268687/ for
>>> another configure patch waiting for a commit.
>>>
>>> Regards,
>>> Stefan
>>>
>>>
>>
>> Ping on getting this into master (and then over to stable).
>>
>
> Thanks Doug.  Anthony, Aurelien, can you commit it?

Ping -- looks like this one got dropped :-(

-- PMM

Patch

diff --git a/configure b/configure
index 2206a87..19d0856 100755
--- a/configure
+++ b/configure
@@ -1386,39 +1386,27 @@  feature_not_found() {
       "configure was not able to find it"
 }
 
-if test -z "$cross_prefix" ; then
-
 # ---
 # big/little endian test
 cat > $TMPC << EOF
-#include <inttypes.h>
-int main(void) {
-        volatile uint32_t i=0x01234567;
-        return (*((uint8_t*)(&i))) == 0x67;
+short big_endian[] = { 0x4269, 0x4765, 0x4e64, 0x4961, 0x4e00, 0, };
+short little_endian[] = { 0x694c, 0x7454, 0x654c, 0x6e45, 0x6944, 0x6e41, 0, };
+extern int foo(short *, short *);
+int main(int argc, char *argv[]) {
+    return foo(big_endian, little_endian);
 }
 EOF
 
-if compile_prog "" "" ; then
-$TMPE && bigendian="yes"
-else
-echo big/little test failed
-fi
-
-else
-
-# 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
+if compile_object ; then
+    if grep -q BiGeNdIaN $TMPO ; then
+        bigendian="yes"
+    elif grep -q LiTtLeEnDiAn $TMPO ; then
+        bigendian="no"
+    else
+        echo big/little test failed
     fi
-  ;;
-  hppa|m68k|mips|mips64|ppc|ppc64|s390|s390x|sparc|sparc64)
-    bigendian=yes
-  ;;
-esac
-
+else
+    echo big/little test failed
 fi
 
 ##########################################