[FFmpeg-devel] [PATCH]Use pushfl/popfl in cpuid.c

Michael Niedermayer michaelni
Sun Nov 23 19:34:49 CET 2008


On Sun, Nov 23, 2008 at 06:09:40PM +0100, Aurelien Jacobs wrote:
> On Sun, 23 Nov 2008 19:52:42 +0100 (CET)
> Carl Eugen Hoyos <cehoyos at rainbow.studorg.tuwien.ac.at> wrote:
> 
> > New try to fix compilation with icc 11.0
> > 
> > Please comment, Carl Eugen
> 
> This most certainly won't work on x86_64 (I will test in
> a few hours if needed).
> 
> Here you have:
>   pushfl
>   pop %0
> where %0 points to a x86_reg variable.
> So pushfl will always be 32 bits while pop will be

my docs say pushfd (pushfl why in ... is (g)as renaming half of the
opcodes, not to mention how wrong the (l)ong postfix is for 32bit on
a long==64bit arch)
doesnt exist on x86_64 in 64bit mode, theres just 16 and 64bit

so it seems we are back to .byte 0x9C

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081123/8039826c/attachment.pgp>



More information about the ffmpeg-devel mailing list