Re: [PATCH] powerpc: Automatically guess --with-cpu value

["Tulio Magno Quites Machado Filho" <tuliom at linux dot vnet dot ibm dot com>]

Almost there.  ;-)

Carlos Eduardo Seo <cseo@linux.vnet.ibm.com> writes:

> diff --git a/sysdeps/powerpc/preconfigure b/sysdeps/powerpc/preconfigure
> index 0c6fdde..4f91add 100644
> --- a/sysdeps/powerpc/preconfigure
> +++ b/sysdeps/powerpc/preconfigure
> @@ -15,3 +15,57 @@ powerpc*)
>    rm -f conftest.i
>    ;;
>  esac
> +
> +# Lets ask the compiler which Power processor we've got, in case the user did
> +# not choose a --with-cpu value.
> +# Note if you add patterns here you must ensure that an appropriate directory
> +# exists in sysdeps/powerpc
> +if test -z "$submachine"; then

$submachine will be "" when the user pass --with-cpu=no, which means the user
wants to force the default submachine.
So, I suspect the right variable is $with_cpu.

> +# Set --with-cpu as appropriate.
> +  case "$archcpu" in
> +  _ARCH_PWR8)
> +    submachine="power8"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR7)
> +    submachine="power7"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR6X)
> +    submachine="power6x"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR6)
> +    submachine="power6"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR5X)
> +    submachine="power5+"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR5)
> +    submachine="power5"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  _ARCH_PWR4)
> +    submachine="power4"
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
> +$as_echo "$as_me:  CPU identified as $submachine" >&6;}
> +    ;;
> +  *)
> +    submachine=""
> +    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU not identified; using default" >&5
> +$as_echo "$as_me:  CPU not identified; using default" >&6;}
> +    ;;
> +  esac

What about this instead?

  case "$archcpu" in
  _ARCH_PWR5X)
    submachine="power5+"
    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
      $as_echo "$as_me:  CPU identified as $submachine" >&6;}
    ;;
  _ARCH_PWR*)
  submachine=$(echo ${archcpu/_ARCH_PWR/power} | tr '[:upper:]' '[:lower:]')
    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU identified as $submachine" >&5
      $as_echo "$as_me:  CPU identified as $submachine" >&6;}
    ;;
  *)
    submachine=""
    { $as_echo "$as_me:${as_lineno-$LINENO}: CPU not identified; using default" >&5
      $as_echo "$as_me:  CPU not identified; using default" >&6;}
    ;;
  esac

-- 
Tulio Magno