From patchwork Fri Jun 18 11:00:23 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: PowerPC64 large toc model Date: Fri, 18 Jun 2010 01:00:23 -0000 From: Alan Modra X-Patchwork-Id: 56173 Message-Id: <20100618110023.GD18443@bubble.grove.modra.org> To: Richard Guenther Cc: David Edelsohn , gcc-patches@gcc.gnu.org On Fri, Jun 18, 2010 at 12:00:29PM +0200, Richard Guenther wrote: > This broke bootstrap on linux64 with > > 11:55 < richi> ../../mem-ref2/gcc/config/rs6000/rs6000.c: In function > 'rs6000_override_options': > 11:55 < richi> ../../mem-ref2/gcc/config/rs6000/rs6000.c:2775:3: error: suggest > braces around empty body in an 'if' statement > [-Werror=empty-body] Oops. You need bleeding edge GNU ld to pass the HAVE_LD_LARGE_TOC configure test, and an in-tree binutils needs to have its version hacked to 2.21. Committing the following as obvious. Sorry about the breakage. * config/rs6000/linux64.h (SET_CMODEL): Don't expand to empty. Index: gcc/config/rs6000/linux64.h =================================================================== --- gcc/config/rs6000/linux64.h (revision 160977) +++ gcc/config/rs6000/linux64.h (working copy) @@ -70,7 +70,7 @@ extern enum rs6000_cmodel cmodel; #define TARGET_CMODEL cmodel #define SET_CMODEL(opt) cmodel = opt #else -#define SET_CMODEL(opt) +#define SET_CMODEL(opt) do {} while (0) #endif #undef PROCESSOR_DEFAULT