[coreboot] [PATCH] romcc --include=settings.h support

[Stefan Reinauer]

On 03.04.2009 15:42 Uhr, Peter Stuge wrote:
> Stefan Reinauer wrote:
>   
>> +++ util/romcc/romcc.c	(working copy)
>> @@ -24998,7 +24998,10 @@
>>  	/* Enter the globl definition scope */
>>  	start_scope(&state);
>>  	register_builtins(&state);
>> +
>>  	compile_file(&state, filename, 1);
>> +	if (includefile)
>> +		compile_file(&state, includefile, 1);
>>     
>
> Shouldn't the include be compiled before the actual file?
>   

Yes, but romcc seems to parse it the other way round. My tests showed
the above produces correct results, while turning the statements around
does not (the first version of my patch looked that way)

Stefan

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090403/e164fd37/attachment.html>