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

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed Dec 10 12:30:19 CET 2008


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.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list