[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