[coreboot] [PATCH]libpayload: add defines to lpgcc

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Jun 24 16:26:20 CEST 2010


On 23.06.2010 14:52, ron minnich wrote:
> On Wed, Jun 23, 2010 at 5:46 AM, Stefan Reinauer <stepan at coresystems.de> wrote:
>   
>> On 6/22/10 5:16 PM, Patrick Georgi wrote:
>>
>>> attached patch tells lpgcc to add two defines to the compiler it
>>> invokes: __LIBPAYLOAD__ and an architecture specific one (__i386__ or
>>> __powerpc__)
>>
>> Isn't __i386__ or __powerpc__ already set by the compiler?
>>     
> Yes it is. Or something is. You should use what the compiler sets IMHO.
>   

I just checked and my gcc 4.2.1 always knows the following #defines on
i386, even if you specify -nostdlib -nostdinc.
#define __i386 1
#define __i386__ 1
#define i386 1

Patrick, which gcc version needs the additional __i386__ #define?

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list