[libav-devel] [PATCH v3 1/2] configure: add support for neon intrinsics

Diego Biurrun diego at biurrun.de
Thu Jun 19 18:55:45 CEST 2014


On Thu, Jun 19, 2014 at 12:29:03PM +0200, Janne Grunau wrote:
> --- a/configure
> +++ b/configure
> @@ -4403,6 +4411,17 @@ enabled_all dxva2 CoTaskMemFree &&
>  
> +for intrin in $INTRINSICS_LIST; do
> +    if enabled $intrin; then
> +        test -n "$intrinsics" &&
> +            die "ERROR: Only one intrinsics instruction set can be enabled." ||
> +            intrinsics="${intrin#intrinsics_}"
> +    fi
> +done

Can this ever happen?  And would it be a problem?

> +# prevent empty $intrinsics
> +! enabled_any $INTRINSICS_LIST && intrinsics=no_intrinsics

I think it would be more elegant to set a default value like "none" or
so instead.

Diego


More information about the libav-devel mailing list