[vlc-devel] [PATCH v2 3/9] contrib/bootstrap: add arm64 for android

[Rémi Denis-Courmont]

Le jeudi 9 octobre 2014, 18:45:21 Thomas Guillem a écrit :
> ---
>  contrib/bootstrap | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/contrib/bootstrap b/contrib/bootstrap
> index 387f665..da45f62 100755
> --- a/contrib/bootstrap
> +++ b/contrib/bootstrap
> @@ -215,6 +215,9 @@ check_android_sdk()
>  	add_make "ANDROID_API := ${ANDROID_API}"
>  	[ ${ANDROID_ABI} = "armeabi-v7a" ] && add_make_enabled "HAVE_NEON"
>  	[ ${ANDROID_ABI} = "armeabi-v7a" ] && add_make_enabled "HAVE_ARMV7A"
> +	[ ${ANDROID_ABI} = "arm64-v8a" ] && add_make_enabled "HAVE_NEON"
> +	[ ${ANDROID_ABI} = "arm64-v8a" ] && add_make_enabled "HAVE_ARMV7A"
> +	[ ${ANDROID_ABI} = "arm64-v8a" ] && add_make_enabled "HAVE_ARMV8A"

Hmm well, insofar as ARMv7-A denote support for ARMv7-A assembler and Thumb, 
this is wrong - except perhaps in 32-bits mode.

For NEON, this is debatable. It really boils down to what libav and postproc 
want.

>  	[ ${ANDROID_ABI} = "armeabi" -a -z "${NO_ARMV6}" ] && add_make_enabled
> "HAVE_ARMV6" }
> 
> @@ -250,6 +253,9 @@ case "${OS}" in
>  			*arm*)
>  			add_make "PLATFORM_SHORT_ARCH := arm"
>  			;;
> +			*arm64*)
> +			add_make "PLATFORM_SHORT_ARCH := arm64"
> +			;;
>  			*i686*)
>  			add_make "PLATFORM_SHORT_ARCH := x86"
>  			;;

-- 
Rémi Denis-Courmont
http://www.remlab.net/