Re: [Qemu-devel] [PATCH 1/9] target-sh4: switch sh4 to softfloat

[Aurelien Jarno]

On Tue, Jan 11, 2011 at 03:35:33PM -0600, Peter Maydell wrote:
> On 11 January 2011 15:22, Nathan Froyd <address@hidden> wrote:
> > On Tue, Jan 11, 2011 at 10:01:30PM +0100, Aurelien Jarno wrote:
> >>  case "$target_arch2" in
> >> -  
> >> alpha|arm|armeb|m68k|microblaze|mips|mipsel|mipsn32|mipsn32el|mips64|mips64el|ppc|ppc64|ppc64abi32|ppcemb|s390x|sparc|sparc64|sparc32plus)
> >> +  
> >> alpha|arm|armeb|m68k|microblaze|mips|mipsel|mipsn32|mipsn32el|mips64|mips64el|ppc|ppc64|ppc64abi32|ppcemb|s390x|sh4|sh4eb|sparc|sparc64|sparc32plus)
> >>      echo "CONFIG_SOFTFLOAT=y" >> $config_target_mak
> >>      ;;
> >>    *)
> >
> > This is obvious, I think...but it's also a little ridiculous.  Why not
> > make everything use softfloat and dispense with this?  How much work
> > would it be on the x86 side?
> 
> If we don't want to do that I guess we could at least change to
> 
> case "$target_arch2" in
>   i386|x86_64|cris)    # I think this list is right :-)

AFAIK, cris doesn't use fpu functions at all, so it can be in both side
without any visible effects.

>     echo "CONFIG_NOSOFTFLOAT=y" >> $config_target_mak
>     ;;
>   *)
>     echo "CONFIG_SOFTFLOAT=y" >> $config_target_mak
>     ;;
> esac
> 
> ?
> 
> -- PMM
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
address@hidden                 http://www.aurel32.net