This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] fix cygwin builds
- From: Jim Wilson <wilson at tuliptree dot org>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 30 Oct 2017 08:41:01 -0700
- Subject: Re: [PATCH] fix cygwin builds
- Authentication-results: sourceware.org; auth=none
- References: <1509328976.2561.16.camel@tuliptree.org> <87y3nt9krv.fsf@linux-m68k.org>
On Mon, 2017-10-30 at 08:07 +0100, Andreas Schwab wrote:
> This broke ia64:
>
> In file included from ./tm_p.h:4:0,
> from ../../gcc/gimplify.c:30:
> ../../gcc/config/ia64/ia64-protos.h:49:13: error: use of enum
> 'memmodel' without previous declaration
> enum memmodel);
> ^
Looks like alpha, sparc, and tilegx have the same problem. Fixed by
including memmodel.h before tm_p.h. Tested with quick x86_64-linux C
only bootstrap, and checked in under the obvious rule.
Jim
2017-10-30 Jim Wilson <wilson@tuliptree.org>
gcc/
* gimplify.c: Include memmodel.h.
Index: gcc/gimplify.c
===================================================================
--- gcc/gimplify.c (revision 254222)
+++ gcc/gimplify.c (working copy)
@@ -27,6 +27,7 @@ along with GCC; see the file COPYING3. If not see
#include "target.h"
#include "rtl.h"
#include "tree.h"
+#include "memmodel.h"
#include "tm_p.h"
#include "gimple.h"
#include "gimple-predict.h"