[coreboot] [PATCH] v2: Enable -O2 / -mcpu=p2 for all 440BX romcc boards
c-d.hailfinger.devel.2006 at gmx.net
Wed Dec 10 15:41:10 CET 2008
On 10.12.2008 14:32, Eric W. Biederman wrote:
> 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.
Thanks for confirming this!
More information about the coreboot