[FFmpeg-devel] [PATCH 0/7] [RFC] x86 assembly constants

James Darnley james.darnley at gmail.com
Sat Oct 3 20:46:25 CEST 2015


On 2015-10-03 04:08, Ronald S. Bultje wrote:
> Hi,
> 
> On Fri, Oct 2, 2015 at 4:58 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
> 
>> On Fri, Oct 2, 2015 at 7:16 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
>>> On Fri, Oct 2, 2015 at 10:08 AM James Darnley <james.darnley at gmail.com>
>>> wrote:
>>>
>>>> The third patch uses them in the remaining inline assembly.
>>>>
>>>
>>> That's the crux of the problem: inline asm uses these constants, but will
>>> be unable to without yasm. Either we drop compatibility for inline asm
>> for
>>> x86 platforms w/o yasm, or we can't do this.
>>>
>>
>> A build without yasm is gimped as it is, so disabling inline asm in
>> the same go doesn't seem like a too terrible thing.
> 
> 
> I'm leaning towards this as well.

Then you will all be pleased to hear that I have fixed building with
--disable-yasm by adding a HAVE_YASM check to a few functions in cavs
(the Chinese H.264 knockoff) and many functions in vc1.  One conditional
in inline_asm has also been extended.  At least this fixes building
ffmpeg for the people who use --disable-yasm.

As for porting, I know I said "how hard can this be"...quite a lot
actually.  I ported 1 function in cavs last night but after going though
vc1 to fix building I can see just how much work that would be.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 603 bytes
Desc: OpenPGP digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151003/52e9c699/attachment.sig>


More information about the ffmpeg-devel mailing list