From: Mathias Krause Date: Wed, 9 Mar 2011 10:30:55 +0000 (+0100) Subject: inteltool: Fixed building of position independent executables X-Git-Url: http://wien.tomnetworks.com/gitweb/?p=coreboot.git;a=commitdiff_plain;h=5782fee0e14557963149c47ad07cf1d235196f67 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 --- 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"