[coreboot] [PATCH] v2: Enable -O2 / -mcpu=p2 for all 440BX romcc boards

[Eric W. Biederman]

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.

Eric