From 693315160e84988028abca5ca6c1d311a479717a Mon Sep 17 00:00:00 2001 From: Damien Zammit Date: Sat, 2 Sep 2017 20:30:39 +1000 Subject: [PATCH] cpu/x86/sipi_vector.S: Use correct op suffix clang wont compile `cmp` asm opcode because it's ambiguous, use the correct op suffix `cmpl` Change-Id: I82da5a9065b382e182dc7d502c7dca2fc717543b Signed-off-by: Damien Zammit Reviewed-on: https://review.coreboot.org/21359 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/cpu/x86/sipi_vector.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cpu/x86/sipi_vector.S b/src/cpu/x86/sipi_vector.S index 1abf3f5d3b..b40ea76f76 100644 --- a/src/cpu/x86/sipi_vector.S +++ b/src/cpu/x86/sipi_vector.S @@ -132,7 +132,7 @@ _start: jnz microcode_done /* Determine if parallel microcode loading is allowed. */ - cmp $0xffffffff, microcode_lock + cmpl $0xffffffff, microcode_lock je load_microcode /* Protect microcode loading. */ @@ -153,7 +153,7 @@ load_microcode: popa /* Unconditionally unlock microcode loading. */ - cmp $0xffffffff, microcode_lock + cmpl $0xffffffff, microcode_lock je microcode_done xor %eax, %eax