Commit a98caf2b authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Khalid Elmously

x86: math-emu: Fix up 'cmp' insn for clang ias

BugLink: https://bugs.launchpad.net/bugs/1889928

[ Upstream commit 81e96851 ]

The clang integrated assembler requires the 'cmp' instruction to
have a length prefix here:

arch/x86/math-emu/wm_sqrt.S:212:2: error: ambiguous instructions require an explicit suffix (could be 'cmpb', 'cmpw', or 'cmpl')
 cmp $0xffffffff,-24(%ebp)
 ^

Make this a 32-bit comparison, which it was clearly meant to be.
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Reviewed-by: default avatarNick Desaulniers <ndesaulniers@google.com>
Link: https://lkml.kernel.org/r/20200527135352.1198078-1-arnd@arndb.deSigned-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarKamal Mostafa <kamal@canonical.com>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
parent 427bce33
...@@ -208,7 +208,7 @@ sqrt_stage_2_finish: ...@@ -208,7 +208,7 @@ sqrt_stage_2_finish:
#ifdef PARANOID #ifdef PARANOID
/* It should be possible to get here only if the arg is ffff....ffff */ /* It should be possible to get here only if the arg is ffff....ffff */
cmp $0xffffffff,FPU_fsqrt_arg_1 cmpl $0xffffffff,FPU_fsqrt_arg_1
jnz sqrt_stage_2_error jnz sqrt_stage_2_error
#endif /* PARANOID */ #endif /* PARANOID */
......
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