[coreboot] [PATCH] romcc --include=settings.h support
Peter Stuge
peter at stuge.se
Fri Apr 3 15:54:21 CEST 2009
Stefan Reinauer wrote:
> >> 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)
Hah! Ok. :) I guess it builds some kind of data structure which is
finally processed in reverse order.
Thanks for explaining it - please put a note in the commit message.
Acked-by: Peter Stuge <peter at stuge.se>
More information about the coreboot
mailing list