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

[Peter Stuge]

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