* [PATCH] fix cygwin builds
@ 2017-10-30 2:07 Jim Wilson
2017-10-30 7:15 ` Andreas Schwab
0 siblings, 1 reply; 3+ messages in thread
From: Jim Wilson @ 2017-10-30 2:07 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 351 bytes --]
Cygwin builds were accidentally broken by the patch for PR
sanitize/82517. This added uses of MAX_SUPPORTED_STACK_ALIGNMENT,
which for cygwin expands to a call to ix86_cfun_abi(), and hence we
need an include of tm_p.h which is currently missing.
Tested with a cygwin default languages bootstrap.
Checked in under the obvious rule.
Jim
[-- Attachment #2: Type: text/x-patch, Size: 407 bytes --]
gcc/
* gimplify.c: Include tm_p.h.
Index: gcc/gimplify.c
===================================================================
--- gcc/gimplify.c (revision 254206)
+++ gcc/gimplify.c (working copy)
@@ -27,6 +27,7 @@
#include "target.h"
#include "rtl.h"
#include "tree.h"
+#include "tm_p.h"
#include "gimple.h"
#include "gimple-predict.h"
#include "tree-pass.h" /* FIXME: only for PROP_gimple_any */
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] fix cygwin builds
2017-10-30 2:07 [PATCH] fix cygwin builds Jim Wilson
@ 2017-10-30 7:15 ` Andreas Schwab
2017-10-30 15:46 ` Jim Wilson
0 siblings, 1 reply; 3+ messages in thread
From: Andreas Schwab @ 2017-10-30 7:15 UTC (permalink / raw)
To: Jim Wilson; +Cc: gcc-patches
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);
^
Andreas.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] fix cygwin builds
2017-10-30 7:15 ` Andreas Schwab
@ 2017-10-30 15:46 ` Jim Wilson
0 siblings, 0 replies; 3+ messages in thread
From: Jim Wilson @ 2017-10-30 15:46 UTC (permalink / raw)
To: Andreas Schwab; +Cc: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 570 bytes --]
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
[-- Attachment #2: Type: text/x-patch, Size: 470 bytes --]
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"
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-30 15:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-30 2:07 [PATCH] fix cygwin builds Jim Wilson
2017-10-30 7:15 ` Andreas Schwab
2017-10-30 15:46 ` Jim Wilson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).