* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
@ 2008-05-26 6:53 ` dfranke at gcc dot gnu dot org
2008-05-26 6:58 ` hjl dot tools at gmail dot com
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2008-05-26 6:53 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from dfranke at gcc dot gnu dot org 2008-05-26 06:52 -------
No idea how this can be related, but I already confirmed that r135881 is not
the cause. Working on it.
--
dfranke at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC|franke dot daniel at gmail |dfranke at gcc dot gnu dot
|dot com |org
AssignedTo|unassigned at gcc dot gnu |dfranke at gcc dot gnu dot
|dot org |org
Status|UNCONFIRMED |ASSIGNED
Ever Confirmed|0 |1
Last reconfirmed|0000-00-00 00:00:00 |2008-05-26 06:52:51
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
2008-05-26 6:53 ` [Bug bootstrap/36331] " dfranke at gcc dot gnu dot org
@ 2008-05-26 6:58 ` hjl dot tools at gmail dot com
2008-05-26 7:16 ` dfranke at gcc dot gnu dot org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: hjl dot tools at gmail dot com @ 2008-05-26 6:58 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from hjl dot tools at gmail dot com 2008-05-26 06:58 -------
Revision 135882 is the cause. A simple testcase:
[hjl@gnu-26 gcc]$ cat x.cc
int
main ()
{
#ifndef __GNUG__
#error "This is bad."
#endif
return 0;
}
[hjl@gnu-26 gcc]$ gcc -c x.cc
[hjl@gnu-26 gcc]$ ./xgcc -B./ -c x.cc
x.cc:5:2: error: #error "This is bad."
[hjl@gnu-26 gcc]$
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
2008-05-26 6:53 ` [Bug bootstrap/36331] " dfranke at gcc dot gnu dot org
2008-05-26 6:58 ` hjl dot tools at gmail dot com
@ 2008-05-26 7:16 ` dfranke at gcc dot gnu dot org
2008-05-26 7:17 ` hjl dot tools at gmail dot com
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2008-05-26 7:16 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from dfranke at gcc dot gnu dot org 2008-05-26 07:16 -------
Accidently removed the conditional definition of __GNUG__.
This seems to fix it:
> $ svn diff c-cppbuiltin.c
Index: c-cppbuiltin.c
===================================================================
--- c-cppbuiltin.c (revision 135891)
+++ c-cppbuiltin.c (working copy)
@@ -384,6 +384,8 @@ define__GNUC__ (void)
cpp_define_formatted (parse_in, "__GNUC_MINOR__=%d", minor);
cpp_define_formatted (parse_in, "__GNUC_PATCHLEVEL__=%d", patchlevel);
+ if (c_dialect_cxx ())
+ cpp_define_formatted (parse_in, "__GNUG__=%d", major);
}
/* Define macros used by <stdint.h>. Currently only defines limits
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
` (2 preceding siblings ...)
2008-05-26 7:16 ` dfranke at gcc dot gnu dot org
@ 2008-05-26 7:17 ` hjl dot tools at gmail dot com
2008-05-26 7:21 ` dfranke at gcc dot gnu dot org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: hjl dot tools at gmail dot com @ 2008-05-26 7:17 UTC (permalink / raw)
To: gcc-bugs
------- Comment #4 from hjl dot tools at gmail dot com 2008-05-26 07:17 -------
(In reply to comment #3)
> Accidently removed the conditional definition of __GNUG__.
> This seems to fix it:
>
> > $ svn diff c-cppbuiltin.c
> Index: c-cppbuiltin.c
> ===================================================================
> --- c-cppbuiltin.c (revision 135891)
> +++ c-cppbuiltin.c (working copy)
> @@ -384,6 +384,8 @@ define__GNUC__ (void)
> cpp_define_formatted (parse_in, "__GNUC_MINOR__=%d", minor);
> cpp_define_formatted (parse_in, "__GNUC_PATCHLEVEL__=%d", patchlevel);
>
> + if (c_dialect_cxx ())
> + cpp_define_formatted (parse_in, "__GNUG__=%d", major);
> }
>
> /* Define macros used by <stdint.h>. Currently only defines limits
>
Can you double what else you may have removed by accident? Thanks.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
` (3 preceding siblings ...)
2008-05-26 7:17 ` hjl dot tools at gmail dot com
@ 2008-05-26 7:21 ` dfranke at gcc dot gnu dot org
2008-05-26 7:44 ` dfranke at gcc dot gnu dot org
2008-05-26 10:23 ` dfranke at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2008-05-26 7:21 UTC (permalink / raw)
To: gcc-bugs
------- Comment #5 from dfranke at gcc dot gnu dot org 2008-05-26 07:20 -------
> Can you double what else you may have removed by accident? Thanks.
Did. Nothing else. Function define__GNUC__ defined 4 macros, my initial patch
only 3 :(
Ok to commit?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
` (4 preceding siblings ...)
2008-05-26 7:21 ` dfranke at gcc dot gnu dot org
@ 2008-05-26 7:44 ` dfranke at gcc dot gnu dot org
2008-05-26 10:23 ` dfranke at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2008-05-26 7:44 UTC (permalink / raw)
To: gcc-bugs
------- Comment #6 from dfranke at gcc dot gnu dot org 2008-05-26 07:44 -------
Subject: Bug 36331
Author: dfranke
Date: Mon May 26 07:43:17 2008
New Revision: 135892
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=135892
Log:
2008-05-26 Daniel Franke <franke.daniel@gmail.com>
PR bootstrap/36331
* c-cppbuiltin.c (define__GNUC__): Re-add definition of __GNUG__.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-cppbuiltin.c
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug bootstrap/36331] [4.4 Regression]: Gcc failed to bootstrap
2008-05-26 2:07 [Bug bootstrap/36331] New: [4.4 Regression]: Gcc failed to bootstrap hjl dot tools at gmail dot com
` (5 preceding siblings ...)
2008-05-26 7:44 ` dfranke at gcc dot gnu dot org
@ 2008-05-26 10:23 ` dfranke at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2008-05-26 10:23 UTC (permalink / raw)
To: gcc-bugs
------- Comment #7 from dfranke at gcc dot gnu dot org 2008-05-26 10:22 -------
Fixed in trunk. Closing.
--
dfranke at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
Target Milestone|--- |4.4.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36331
^ permalink raw reply [flat|nested] 8+ messages in thread