[FFmpeg-devel] [PATCH 1/2] configure: add check for AVX inline support

Ronald S. Bultje rsbultje at gmail.com
Fri May 16 13:49:24 CEST 2014


Hi,

On Thu, May 15, 2014 at 11:42 PM, Michael Niedermayer <michaelni at gmx.at>wrote:

> On Thu, May 15, 2014 at 07:03:02PM -0300, James Almer wrote:
> > Signed-off-by: James Almer <jamrial at gmail.com>
> > ---
> >  configure | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
>
> applied


I have big objections to this. Inline is unreadable, unportable (e.g.
doesn't work on MSVC) and virtually nobody understands inline. It's beyond
me that anyone wants to write avx in this atrocity of a syntax.

Can we please revert this and rewrite patch 2/2 in yasm syntax? I would be
greatly thankful.

Thanks,
Ronald


More information about the ffmpeg-devel mailing list