AARCH64: RFR (XS) 8068927: AARCH64: better handling of aarch64- triples (was RFR: AARCH64: Top-level JDK changes)

[Dean Long]

Can I get a review for this?

https://bugs.openjdk.java.net/browse/JDK-8068927
http://cr.openjdk.java.net/~dlong/8068927/webrev/

thanks,

dl

On 1/13/2015 10:56 AM, Dean Long wrote:
> On 1/13/2015 1:08 AM, Andrew Haley wrote:
>> On 13/01/15 08:44, Dean Long wrote:
>>
>>> I came up with a simpler version, where I replace "aarch64-" with
>>> "arm-", run autoconf-config.sub, then replace "arm-" back to
>>> "aarch64-".
>> Thanks.  That sounds good to me.
>>
>> Andrew.
>
> Here's the patch.  If it looks good, I can file a bug and push it to 
> the staging repo.
>
> dl
>
>
> diff -r b052cb38b985 common/autoconf/build-aux/config.sub
> --- a/common/autoconf/build-aux/config.sub      Thu Dec 11 15:05:06 
> 2014 -0800
> +++ b/common/autoconf/build-aux/config.sub      Tue Jan 13 13:57:23 
> 2015 -0500
> @@ -41,25 +41,8 @@
>      case $1 in
>          -- )   # Stop option processing
>              shift; break ;;
> -        aarch64-gnu )
> -            sub_args="$sub_args aarch64-unknown-gnu"
> -            shift; ;;
> -        aarch64-linux )
> -            sub_args="$sub_args aarch64-unknown-linux-gnu"
> -            shift; ;;
> -        aarch64-*-linux )
> -            os=`echo $1 | sed 's/aarch64-\(.*\)-linux/\1/'`
> -            config="aarch64-unknown-linux-gnu"
> -            sub_args="$sub_args $config"
> -            shift; ;;
> -        aarch64-*-gnu )
> -            os=`echo $1 | sed 's/aarch64-\(.*\)-gnu.*$/\1/'`
> -            config="aarch64-unknown-gnu"
> -            sub_args="$sub_args $config"
> -            shift; ;;
> -        aarch64-*-linux-* )
> -            os=`echo $1 | sed 's/aarch64-\(.*\)-linux-.*$/'`
> -            config="aarch64-unknown-linux-gnu"
> +        aarch64-* )
> +            config=`echo $1 | sed 's/^aarch64-/arm-/'`
>              sub_args="$sub_args $config"
>              shift; ;;
>          - )    # Use stdin as input.
> @@ -74,9 +57,7 @@
>  result=`. $DIR/autoconf-config.sub $sub_args "$@"`
>  exitcode=$?
>
> -if [ "x$os" != "x" ] ; then
> -    result=`echo $result | sed "s/-unknown-/-$os-/"`
> -fi
> +result=`echo $result | sed "s/^arm-/aarch64-/"`
>
>  echo $result
>  exit $exitcode
>