[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread howarth at nitro dot med.uc.edu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

Jack Howarth howarth at nitro dot med.uc.edu changed:

   What|Removed |Added

 CC||howarth at nitro dot
   ||med.uc.edu

--- Comment #1 from Jack Howarth howarth at nitro dot med.uc.edu 2011-10-17 
14:34:30 UTC ---
I don't see this issue at r180087 on x86_64-apple-darwin11 with...

../gcc-4.7-20111017/configure --prefix=/sw --prefix=/sw/lib/gcc4.7
--mandir=/sw/share/man --infodir=/sw/lib/gcc4.7/info
--with-build-config=bootstrap-lto --enable-stage1-languages=c,lto
--enable-languages=c,c++,fortran,lto,objc,obj-c++,java --with-gmp=/sw
--with-libiconv-prefix=/sw --with-ppl=/sw --with-cloog=/sw --with-mpc=/sw
--with-system-zlib --x-includes=/usr/X11R6/include --x-libraries=/usr/X11R6/lib
--program-suffix=-fsf-4.7 --enable-checking=yes --enable-cloog-backend=isl

and an lto-bootstrap.


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

Richard Guenther rguenth at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2011-10-17
   Target Milestone|--- |4.7.0
 Ever Confirmed|0   |1

--- Comment #2 from Richard Guenther rguenth at gcc dot gnu.org 2011-10-17 
15:12:20 UTC ---
Confirmed.


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

--- Comment #3 from dodji at seketeli dot org dodji at seketeli dot org 
2011-10-17 15:26:54 UTC ---
Are you still seing this with commit r180090?


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

--- Comment #4 from Dominique d'Humieres dominiq at lps dot ens.fr 2011-10-17 
15:42:09 UTC ---
 Are you still seing this with commit r180090?

I have bootstrapped revision 180087 with the following patch

--- /opt/gcc/_clean/libcpp/line-map.c2011-10-17 12:04:41.0 +0200
+++ /opt/gcc/work/libcpp/line-map.c2011-10-17 15:36:30.0 +0200
@@ -625,7 +625,7 @@ source_location
 linemap_macro_map_loc_to_exp_point (const struct line_map *map,
 source_location location)
 {
-  unsigned token_no;
+  unsigned token_no __attribute__ ((__unused__));

   linemap_assert (linemap_macro_expansion_map_p (map)
location = MAP_START_LOCATION (map));

I guess that revision 180090 is a similar fix that should work (I'll have the
answer in a couple hours;-).

Last question: is the warning bogus or not? If no, why? If yes, I'll open a new
PR.


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

--- Comment #5 from dodji at seketeli dot org dodji at seketeli dot org 
2011-10-17 15:56:05 UTC ---
dominiq at lps dot ens.fr gcc-bugzi...@gcc.gnu.org a écrit:

 I guess that revision 180090 is a similar fix that should work (I'll
 have the answer in a couple hours;-).

OK, thanks.

 Last question: is the warning bogus or not? If no, why? If yes, I'll
 open a new PR.

The warning is not bogus when you are compiling with --disable-checking
or when your GCC_VERSION is  2007.  This is because the way
linemap_assert is defined:

/* Assertion macro to be used in line-map code.  */
#define linemap_assert(EXPR)\
  do {\
if (! (EXPR))\
  abort ();\
  } while (0)

/* Assert that MAP encodes locations of tokens that are not part of
   the replacement-list of a macro expansion.  */
#define linemap_check_ordinary(LINE_MAP) __extension__\
  ({linemap_assert (!linemap_macro_expansion_map_p (LINE_MAP)); \
(LINE_MAP);})
#else
#define linemap_assert(EXPR)
#define linemap_check_ordinary(LINE_MAP) (LINE_MAP)
#endif

In those cases token_no is not used.  Hence the fix committed to
revision 180090.

Sorry for the inconvenience.


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

--- Comment #6 from Dominique d'Humieres dominiq at lps dot ens.fr 2011-10-17 
19:40:06 UTC ---
 The warning is not bogus when you are compiling with --disable-checking
 or when your GCC_VERSION is  2007.  ...

I am using

../work/configure --prefix=/opt/gcc/gcc4.7w
--enable-languages=c,c++,fortran,objc,obj-c++,java,ada,lto --with-gmp=/opt/sw64
--with-libiconv-prefix=/opt/sw64 --with-system-zlib --with-cloog=/opt/sw64
--enable-cloog-backend=isl --enable-lto

so no --disable-checking and the bootstrap gcc is

configure:4007: gcc --version 5
gcc (GCC) 4.5.3 20110419 for GNAT GPL 2011 (20110419)

What am I missing?


[Bug bootstrap/50758] [4.7 Regression] Bootstrap fails at stage 2 on x86_64-apple-darwin10: error: variable 'token_no' set but not used

2011-10-17 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50758

Dominique d'Humieres dominiq at lps dot ens.fr changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #7 from Dominique d'Humieres dominiq at lps dot ens.fr 2011-10-17 
19:56:33 UTC ---
BTW bootstrap finished on x86_64-apple-darwin10 and line-map.c compiled at
stage2 on powerpc-apple-darwin9, both at revision 180100. Thanks for the patch.
Closing as fixed.