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

[Carl-Daniel Hailfinger]

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

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 
pending an answer from Eric.

Regards,
Carl-Daniel