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 <[email protected]>

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to