Hi Loren,

On Sat, Jun 15, 2013 at 5:53 PM, Loren Merritt <[email protected]>wrote:

> 1.5x-1.8x faster on sandybridge
> ---
>  libavutil/lls.c          |  3 +++
>  libavutil/lls.h          |  1 +
>  libavutil/x86/lls.asm    | 31 +++++++++++++++++++++++++++++++
>  libavutil/x86/lls_init.c |  6 +++++-
>  4 files changed, 40 insertions(+), 1 deletion(-)
>
> diff --git a/libavutil/lls.c b/libavutil/lls.c
> index eb500af..8f1aff1 100644
> --- a/libavutil/lls.c
> +++ b/libavutil/lls.c
> @@ -119,6 +119,9 @@ double avpriv_evaluate_lls(LLSModel *m, double *param,
> int order)
>      int i;
>      double out = 0;
>
> +    if (m->evaluate_lls)
> +        return m->evaluate_lls(m->coeff[order], param, order);
>

Is there a special reason you didn't assign the default code as default
implementation for evaluate_lls (as in: evaluate_lls_c), as is commonly
done?

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

Reply via email to