[Bug c++/45588] unused-but-set-variable false trigger building gold

2010-09-09 Thread jakub at gcc dot gnu dot org 


--- Comment #4 from jakub at gcc dot gnu dot org  2010-09-09 06:51 ---
Subject: Bug 45588

Author: jakub
Date: Thu Sep  9 06:50:56 2010
New Revision: 164051

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164051
Log:
PR c++/45588
* pt.c (tsubst) case INTEGER_TYPE: Call mark_rvalue_use
before calling fold_decl_constant_value.

* g++.dg/warn/Wunused-var-15.C: New test.

Added:
trunk/gcc/testsuite/g++.dg/warn/Wunused-var-15.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/pt.c
trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45588

[Bug c++/45588] unused-but-set-variable false trigger building gold

2010-09-09 Thread jakub at gcc dot gnu dot org 


--- Comment #5 from jakub at gcc dot gnu dot org  2010-09-09 07:16 ---
Fixed.


-- 

jakub at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45588

[Bug c++/45588] unused-but-set-variable false trigger building gold

2010-09-07 Thread amodra at gmail dot com 


--- Comment #1 from amodra at gmail dot com  2010-09-07 21:59 ---
Confirmed on powerpc-linux 4.6.0 20100905
$ ~/build/ppc/gcc-curr/gcc/g++ -B ~/build/ppc/gcc-curr/gcc/ -I
~/build/ppc/gcc-curr/powerpc-linux/libstdc++-v3/include/powerpc-linux -I
~/build/ppc/gcc-curr/powerpc-linux/libstdc++-v3/include -I
/src/gcc-current/libstdc++-v3/libsupc++ -DHAVE_CONFIG_H -I.
-I/src/binutils-current/gold  -I/src/binutils-current/gold
-I/src/binutils-current/gold/../include -I/src/binutils-current/gold/../elfcpp
-DLOCALEDIR=\/usr/local/share/locale\ -DBINDIR=\/usr/local/bin\
-DTOOLBINDIR=\/usr/local/powerpc-linux/bin\   -W -Wall-Werror
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -frandom-seed=plugin.o -g -O
-fno-inline -msecure-plt -MT plugin.o -MD -MP -MF .deps/plugin.Tpo -c -o
plugin.o /src/binutils-current/gold/plugin.cc
/src/binutils-current/gold/plugin.cc: In member function 'void
gold::Sized_pluginobjsize, big_endian::do_add_symbols(gold::Symbol_table*,
gold::Read_symbols_data*, gold::Layout*) [with int size = 64, bool big_endian =
false]':
/src/binutils-current/gold/plugin.cc:1166:1:   instantiated from here
[snip]


-- 

amodra at gmail dot com changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-09-07 21:59:46
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45588

[Bug c++/45588] unused-but-set-variable false trigger building gold

2010-09-07 Thread paolo dot carlini at oracle dot com 


--- Comment #2 from paolo dot carlini at oracle dot com  2010-09-07 22:28 
---
Let's add Jakub in CC.


-- 

paolo dot carlini at oracle dot com changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45588

[Bug c++/45588] unused-but-set-variable false trigger building gold

2010-09-07 Thread jakub at gcc dot gnu dot org 


--- Comment #3 from jakub at gcc dot gnu dot org  2010-09-07 22:45 ---
Reproduced with:
void bar (unsigned char *);

template int N
struct S
{
  static const int k = 6;
};

template int N
const int SN::k;

template int N
void foo ()
{
  const int i = SN::k;
  unsigned char a[i];
  bar (a);
}

void bar ()
{
  foo64 ();
}


-- 

jakub at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-09-07 21:59:46 |2010-09-07 22:45:05
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45588