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

[Ronald S. Bultje]

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