Patchwork PowerPC64 large toc model

login
register
mail settings
Submitter Alan Modra
Date June 18, 2010, 11 a.m.
Message ID <20100618110023.GD18443@bubble.grove.modra.org>
Download mbox | patch
Permalink /patch/56173/
State New
Headers show

Comments

Alan Modra - June 18, 2010, 11 a.m.
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.

Patch

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