[coreboot] [PATCH] v2: Enable -O2 / -mcpu=p2 for all 440BX romcc boards
Eric W. Biederman
ebiederm at xmission.com
Wed Dec 10 14:32:41 CET 2008
Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> writes:
> Hi Eric,
> are there any reasons (bugs) to avoid using -O2 for ROMCC?
> On 10.12.2008 11:19, Uwe Hermann wrote:
>> See patch.
>> Build-tested and tested on hardware using an ASUS P2B-F, I didn't notice
>> any problems. However, if there are any non-obvious reasons to _not_
>> enable optimizations in romcc, please let me know.
>> Use -O2 and -mcpu=p2 as romcc options for all Intel 440BX boards.
>> This should hopefully make the "too few registers" error pop up less often.
>> Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
> I'd prefer to switch all these boards to CAR, but as this patch improves
> the current situation, it is
> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
> pending an answer from Eric.
I want to say to my knowledge all of the failures in romcc kill the build.
I can remember one very rare case where that was not true.
-mcpu=p2 should always be safe, and I would not expect any problems with -O2.
More information about the coreboot