Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

[Andrei Borzenkov]

19.08.2015 21:35, Mark H Weaver пишет:
> Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile
> code for the O32 ABI when targetting MIPS, since the MIPS assembly code
> in GRUB assumes this.  This flag is also needed when compiling
> asm-tests/mips.S from configure, because GNU as rejects MIPS register
> names such as $t2 unless the O32 ABI is selected.
> ---
>   conf/Makefile.common | 4 ++++
>   configure.ac         | 7 +++++--
>   2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/conf/Makefile.common b/conf/Makefile.common
> index fcb8d2e..bd125da 100644
> --- a/conf/Makefile.common
> +++ b/conf/Makefile.common
> @@ -20,6 +20,10 @@ endif
>   if COND_powerpc_ieee1275
>     CFLAGS_PLATFORM += -mcpu=powerpc
>   endif
> +if COND_mips
> +  CFLAGS_PLATFORM += -mabi=32
> +  CCASFLAGS_PLATFORM = -mabi=32
> +endif

Does it work with clang?

>   #FIXME: discover and check XEN headers
>   CPPFLAGS_XEN = -I/usr/include
> diff --git a/configure.ac b/configure.ac
> index c864311..1f5e8a2 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2,7 +2,8 @@
>
>   # Process this file with autoconf to produce a configure script.
>
> -# Copyright (C) 2002,2003,2004,2005,2006,2007,2008,2009,2010  Free Software 
> Foundation, Inc.
> +# Copyright (C) 2002,2003,2004,2005,2006,2007,2008,2009,2010,2011,
> +#   2012,2013,2014,2015  Free Software Foundation, Inc.
>   #
>   # This configure.ac is free software; the author
>   # gives unlimited permission to copy and/or distribute it,
> @@ -599,9 +600,11 @@ fi
>
>   AC_CACHE_CHECK([for options to compile assembly], 
> [grub_cv_cc_target_asm_compile], [
>   test_program=
> +test_ccasflags=
>   case "x$target_cpu-$platform" in
>        xmips-* | xmipsel-*)
>           test_program=mips
> +        test_ccasflags=-mabi=32
>         ;;
>        xi386-pc)
>          test_program=i386-pc
> @@ -618,7 +621,7 @@ if test x"$test_program" = x ; then
>   else
>     found=no
>     for arg in "" "-no-integrated-as"; do
> -    cmdline="$TARGET_CC -c -o /dev/null $TARGET_CCASFLAGS $arg $TARGET_CPPFLAGS 
> $srcdir/asm-tests/$test_program.S"
> +    cmdline="$TARGET_CC -c -o /dev/null $TARGET_CCASFLAGS $test_ccasflags $arg 
> $TARGET_CPPFLAGS $srcdir/asm-tests/$test_program.S"
>       echo "Running $cmdline" >&AS_MESSAGE_LOG_FD
>       if $cmdline >&AS_MESSAGE_LOG_FD 2>&AS_MESSAGE_LOG_FD; then
>         grub_cv_cc_target_asm_compile="$arg"