On Tue, May 7, 2013 at 7:29 PM, Steve Ellcey wrote: > On Tue, 2013-05-07 at 19:16 +0200, Steven Bosscher wrote: > >> On Tue, May 7, 2013 at 7:13 PM, Graham Stott wrote: >> > Steven, >> > >> > The new execute mips_machine_reorrg2 part of the patch doesn't have a >> > return value! >> >> Right, brilliant... Thanks for spotting that. >> >> Steve, can you try this, please? >> >> >> Index: config/mips/mips.c >> =================================================================== >> --- config/mips/mips.c (revision 198640) >> +++ config/mips/mips.c (working copy) >> @@ -16388,6 +16388,7 @@ mips_machine_reorg2 (void) >> optimizations, but this should be an extremely rare case anyhow. */ >> mips_reorg_process_insns (); >> mips16_split_long_branches (); >> + return 0; >> } >> >> struct rtl_opt_pass pass_mips_machine_reorg2 = > > Yes, this fixed my build.
I've committed this, r198696. Ciao! Steven