From 5782fee0e14557963149c47ad07cf1d235196f67 Mon Sep 17 00:00:00 2001 From: Mathias Krause Date: Wed, 9 Mar 2011 11:30:55 +0100 Subject: [PATCH] inteltool: Fixed building of position independent executables When building a position independent executable (PIE) EBX is used internally by the compiler to generate position independent address references so it cannot be used in the clobber list. Use the already existing code for the Darwin plattform for that case, too -- it'll preserve the EBX value. Change-Id: Ief6d4872b8cd990856a0e8227a88bb228782aced Signed-off-by: Mathias Krause Reviewed-on: http://review.coreboot.org/209 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones --- util/inteltool/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/inteltool/cpu.c b/util/inteltool/cpu.c index e79380aef..9037d5df6 100644 --- a/util/inteltool/cpu.c +++ b/util/inteltool/cpu.c @@ -32,7 +32,7 @@ unsigned int cpuid(unsigned int op) { uint32_t ret; -#if defined(__DARWIN__) && !defined(__LP64__) +#if defined(__PIC__) || defined(__DARWIN__) && !defined(__LP64__) asm volatile ( "pushl %%ebx\n" "cpuid\n" -- 2.25.1