Commit 180d43bc authored by Mans Rullgard's avatar Mans Rullgard

x86: use nop cpu directives only if supported

nasm does not support 'CPU foonop' directives.  This adds a configure
test for the directive and uses it only if supported.
Signed-off-by: default avatarMans Rullgard <>
parent 72382650
......@@ -1084,6 +1084,7 @@ HAVE_LIST="
......@@ -2889,6 +2890,7 @@ EOF
die "yasm not found, use --disable-yasm for a crippled build"
check_yasm "vextractf128 xmm0, ymm0, 0" || disable avx
check_yasm "vfmaddps ymm0, ymm1, ymm2, ymm3" || disable fma4
check_yasm "CPU amdnop" && enable cpunop
case "$cpu" in
......@@ -91,8 +91,14 @@
default rel
%macro CPUNOP 1
CPU %1
; Always use long nops (reduces 0x90 spam in disassembly on x86_32)
CPU amdnop
CPUNOP amdnop
; Macros to eliminate most code duplication between x86_32 and x86_64:
; Currently this works only for leaf functions which load all their arguments
......@@ -589,7 +595,7 @@ SECTION .note.GNU-stack noalloc noexec nowrite progbits
; All subsequent functions (up to the next INIT_CPUFLAGS) is built for the specified cpu.
; You shouldn't need to invoke this macro directly, it's a subroutine for INIT_MMX &co.
%macro INIT_CPUFLAGS 0-2
CPU amdnop
CPUNOP amdnop
%if %0 >= 1
%xdefine cpuname %1
%assign cpuflags cpuflags_%1
......@@ -612,7 +618,7 @@ SECTION .note.GNU-stack noalloc noexec nowrite progbits
%define movu lddqu
%if notcpuflag(mmx2)
CPU basicnop
CPUNOP basicnop
%xdefine SUFFIX
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment