On 2014-08-09 08:06:12 -0700, Diego Biurrun wrote:
> ---
> 
> Implemented Janne's suggestion.  Untested since I don't have the right system.
> 
>  configure | 15 ++++++++++++---
>  1 file changed, 12 insertions(+), 3 deletions(-)
> 
> diff --git a/configure b/configure
> index 46b8391..7c2fab6 100755
> --- a/configure
> +++ b/configure
> @@ -2199,8 +2199,6 @@ ranlib="ranlib"
>  strip="strip"
>  yasmexe="yasm"
>  
> -nogas=":"
> -
>  # machine
>  arch_default=$(uname -m)
>  cpu="generic"
> @@ -3420,7 +3418,6 @@ case $target_os in
>          add_extralibs -lpoll -lgnugetopt
>          ;;
>      darwin)
> -        gas="gas-preprocessor.pl $cc"
>          enabled ppc && add_asflags -force_cpusubtype_ALL
>          SHFLAGS='-dynamiclib -Wl,-single_module 
> -Wl,-install_name,$(SHLIBDIR)/$(SLIBNAME_WITH_MAJOR),-current_version,$(LIBVERSION),-compatibility_version,$(LIBMAJOR)'
>          enabled x86_32 && append SHFLAGS -Wl,-read_only_relocs,suppress
> @@ -3779,9 +3776,21 @@ EOF
>  }
>  
>  if enabled_any arm aarch64 || enabled_all ppc altivec && enabled asm; then
> +    nogas=:
>      enabled_any arm aarch64 && nogas=die
>      enabled_all ppc altivec && nogas=warn
>  
> +    case $as_type
> +        arm*) gaspp_as_type=armasm ;;
> +        gcc)  gaspp_as_type=gas ;;
> +        *)    gaspp_as_type=$as_type ;;
> +    esac
> +
> +    [ $target_os = "darwin" ] && gaspp_as_type="apple-$as_type"
> +
> +    check_cmd gas-preprocessor.pl -arch $arch -as-type $gaspp_as_type &&
> +        gas="gas-preprocessor.pl -arch $arch -as-type $gaspp_as_type $as"

missing '--' to divide gaspp options from the assembler cmd line. looks 
good otherwise. I'll test aarch64/arm with Xcode on darwin and llvm on 
linux. Martin can hopefully test armasm.

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to