diff options
| author | Martin Storsjö <martin@martin.st> | 2024-01-26 14:55:49 +0200 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2024-02-09 23:55:52 +0200 |
| commit | 7ec2354c38978b918dc079b611393becb6c80bf7 (patch) | |
| tree | 97ea79f379c8cab30a73f2c690bb76e5dabc7127 /libavcodec/vaapi_encode.h | |
| parent | d5aaed9d4c60a1c3508317bf0cbc469eab339932 (diff) | |
| download | ffmpeg-7ec2354c38978b918dc079b611393becb6c80bf7.tar.gz | |
x86: Remove inline MMX assembly that clobbers the FPU state
These inline implementations of AV_COPY64, AV_SWAP64 and AV_ZERO64
are known to clobber the FPU state - which has to be restored
with the 'emms' instruction afterwards.
This was known and signaled with the FF_COPY_SWAP_ZERO_USES_MMX
define, which calling code seems to have been supposed to check,
in order to call emms_c() after using them. See
0b1972d4096df5879038f0af776f87f41e90ebd4,
29c4c0886d143790fcbeddbe40a23dfc6f56345c and
df215e575850e41b19aeb1fd99e53372a6b3d537 for history on earlier
fixes in the same area.
However, new code can use these AV_*64() macros without knowing
about the need to call emms_c().
Just get rid of these dangerous inline assembly snippets; this
doesn't make any difference for 64 bit architectures anyway.
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'libavcodec/vaapi_encode.h')
0 files changed, 0 insertions, 0 deletions