[FFmpeg-devel] Patch: Inline asm fixes for Intel compiler on Windows

Michael Niedermayer michaelni at gmx.at
Mon Mar 17 01:02:30 CET 2014

On Mon, Mar 17, 2014 at 10:44:56AM +1100, Matt Oliver wrote:
> Heres an update to patch 1 that fixes the warnings (some variables got
> orphaned since i originally wrote the patch).
> Also an updated patch 6 with the "'s removed (they snuck back in when
> rebasing somehow).
> the change to round could be in a separate patch
> The changes to round are required as otherwise the syntax used to apply the
> offset (i.e. 2*16) is incorrect when using named constraints (icl will
> error like crazy). Which is why its in the same patch as otherwise the
> changes wont work anyway and I didnt think a broken patch was all that
> useful. f you want me to separate them then Although if its easier for you
> if I separate them then let me know.

the change to ROUND should be fine before the rest

> So which patches have been applied

9eb3f11 Add missing external declarations.
590805b Fixed 64bit conformance with mvzbl.

> and which do you want me to rework?

some of the patches seem to not apply cleanly


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140317/d9801d9b/attachment.asc>

More information about the ffmpeg-devel mailing list