[FFmpeg-devel] [PATCH]Remove test for cpuid instruction from cpuid.c

Michael Niedermayer michaelni
Sun Nov 23 17:12:54 CET 2008


On Sun, Nov 23, 2008 at 04:06:12PM +0000, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> 
> > On Sun, Nov 23, 2008 at 03:40:05PM +0000, M?ns Rullg?rd wrote:
> >> Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> >> 
> >> > On Sun, Nov 23, 2008 at 01:45:07PM +0000, Carl Eugen Hoyos wrote:
> >> >> I will copy the assembly sources into issue 709: It would be nice
> >> >> if somebody who actually understands the code could comment there.
> >> >
> >> > Please replace pushf/popf by pushfd/popfd.
> >> 
> >> Can someone please explain how the same opcode can behave differently
> >> depending on the assembler mnemonic used?
> >
> > Uh, as I read it ICC compiled pushf to pushfw (16 bit) instead of pushfd
> > (32 bit). At least that was my theory.
> 
> Yes, but all the pushf* instructions have the same opcode (0x9c).

pushf(not current address size) 0x66 0x9C
pushf(current address size)     0x9C

use .byte ...
to convince the crapiler to generate the correct one


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- 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/aeb82686/attachment.pgp>



More information about the ffmpeg-devel mailing list