[RFC] missing return warnings

2007-09-06 Thread Jan Hubicka
Hi, I am looking into what parts of frontend stil depends on frontend inline decision (ie DECL_INLINE now ignored by middle end). All these uses strike wrong, given that inlining decisions are independent on it. These divergences for example requires us to enable expensive -finline-functions for -

Re: GCC "make" errors

2007-09-06 Thread mandeep singh bhambra
I have installed the latest binutils (2.9.1) available from the GNU ftp site so I cannot understand why this is occuring. Are there some sort of parameter options I need to enter or do I need to reinstall the binutils with parameter options? Regards, > - Original Message - > From: "Ric

Re: Someone has caused regressions in gfortran

2007-09-06 Thread Jan Hubicka
> > Sadly, the testsuite regressions don't seems to be fixed. I will try to > > figure out tomorrow why the function is still being inlined. > > The test case gfortran.dg/do_3.F90 pass with -fno-strict-overflow > (see http://gcc.gnu.org/ml/fortran/2007-09/msg00116.html). > I have posted at http:/

Bootstrap failure under Irix

2007-09-06 Thread Philippe Schaffnit
Hi! I am seeing the folowing bootstrap failure under SGI Irix: bash /USER/philippe/Irix/Gcc_Sources/gcc/../move-if-change tmp-codes.h insn-codes.h echo timestamp > s-codes gcc -c -g -fkeep-inline-functions -DIN_GCC -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -Wold-style

Re: Someone has caused regressions in gfortran

2007-09-06 Thread Dominique Dhumieres
> The testcase was indeed previously not inlined at all. Shall we add > -fno-strict-overflow to the testcase then? This what I would do, but I am not qualified to make the call. In addition my working setup is totally broken right now (at stage1). Could you do the addition to the testcase and ru

Bootstrap broken: ../../gcc/gcc/ada/trans.c:5896: error: too few arguments to function `real_2expN'

2007-09-06 Thread Christian Joensson
Using checkout Thu Sep 6 05:56:16 UTC 2007 (revision 128174), I get a bootstrap failure: gcc -c -g -fkeep-inline-functions -DIN_GCC -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -DHAVE_CONFIG_H -I. -Iada -I../../gcc/gcc -I../../gcc/gcc/ada -I../../gcc/gcc/../include -I../.

Bootstrap failure under Linux

2007-09-06 Thread Philippe Schaffnit
Hi! I am getting the following bootstrap failure under Linux (Linux pinguin7 2.6.5-7.104-bigsmp #1 SMP Wed Jul 28 16:42:13 UTC 2004 i686 i686 i386 GNU/Linux). make[2]: Leaving directory `/usr1/MICRESS/Philippe/Compilation/Gcc' make "DESTDIR=" "RPATH_ENVVAR=LD_LIBRARY_PATH" "TARGET_SUBDIR=i686-pc

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Richard Guenther
On 9/5/07, Mark Mitchell <[EMAIL PROTECTED]> wrote: > Summary > === > > We are closing in on Stage 3, previously announced for September 10th. > At this point, I'm not aware of any reason to delay that date. Are > there any Stage 2 patches that people don't think will be submitted by > that po

Re: GCC "make" errors

2007-09-06 Thread Richard Guenther
On 9/6/07, mandeep singh bhambra <[EMAIL PROTECTED]> wrote: > I have installed the latest binutils (2.9.1) available from the GNU ftp site > so I cannot understand why this is occuring. Are there some sort of parameter > options I need to enter or do I need to reinstall the binutils with paramete

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Paolo Bonzini
There is http://gcc.gnu.org/ml/gcc-patches/2007-08/msg01978.html for example, which is not suitable for stage3. As much as I like the idea, wasn't get_non_trapping considered unsafe? Paolo

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Richard Guenther
On 9/6/07, Paolo Bonzini <[EMAIL PROTECTED]> wrote: > > > There is > > > > http://gcc.gnu.org/ml/gcc-patches/2007-08/msg01978.html > > > > for example, which is not suitable for stage3. > > As much as I like the idea, wasn't get_non_trapping considered unsafe? Only if you tried to preserve this in

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Manuel López-Ibáñez
On 05/09/07, Mark Mitchell <[EMAIL PROTECTED]> wrote: > Summary > === > > We are closing in on Stage 3, previously announced for September 10th. > At this point, I'm not aware of any reason to delay that date. Are > there any Stage 2 patches that people don't think will be submitted by > that

Re: Someone has caused regressions in gfortran (c_char_tests_red.f03)

2007-09-06 Thread Dominique Dhumieres
I have done some investigation about the recent failure of gfortran.dg/c_char_tests.f03. First the failure disappears with -fno-inline or -fno-inline-functions: [karma] f90/bug% gfc c_char_tests_db.f03 -O3 -fno-inline c_char_driver_db.c [karma] f90/bug% a.out [karma] f90/bug% gfc c_char_tests_db.f

Re: [RFC] missing return warnings

2007-09-06 Thread Manuel López-Ibáñez
On 06/09/07, Jan Hubicka <[EMAIL PROTECTED]> wrote: > I wonder what we want to do here - I guess we can either make the > warning unconditional and declare it as two indpendent things or we can > just drop the feature since user will get properly warned on every > function he actually uses. > > Wha

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Jan Hubicka
> Summary > === > > We are closing in on Stage 3, previously announced for September 10th. > At this point, I'm not aware of any reason to delay that date. Are > there any Stage 2 patches that people don't think will be submitted by > that point? I am still planning to do some retuning of in

Re: Bootstrap broken: ../../gcc/gcc/ada/trans.c:5896: error: too few arguments to function `real_2expN'

2007-09-06 Thread Sandra Loosemore
Christian Joensson wrote: Using checkout Thu Sep 6 05:56:16 UTC 2007 (revision 128174), I get a bootstrap failure: gcc -c -g -fkeep-inline-functions -DIN_GCC -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -DHAVE_CONFIG_H -I. -Iada -I../../gcc/gcc -I../../gcc/gcc/ada -I../

Re: Bootstrap broken: ../../gcc/gcc/ada/trans.c:5896: error: too few arguments to function `real_2expN'

2007-09-06 Thread Christian Joensson
2007/9/6, Sandra Loosemore <[EMAIL PROTECTED]>: > Christian Joensson wrote: > > Using checkout Thu Sep 6 05:56:16 UTC 2007 (revision 128174), I get a > > bootstrap failure: > > > > gcc -c -g -fkeep-inline-functions -DIN_GCC -W -Wall > > -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes

Re: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Sebastian Pop
On 9/4/07, Mark Mitchell <[EMAIL PROTECTED]> wrote: > We are closing in on Stage 3, previously announced for September 10th. > At this point, I'm not aware of any reason to delay that date. Are > there any Stage 2 patches that people don't think will be submitted by > that point? > I still plan t

Re: GCC "make" errors

2007-09-06 Thread Tim Prince
mandeep singh bhambra wrote: > I have installed the latest binutils (2.9.1) available from the GNU ftp site > so I cannot understand why this is occuring. Are there some sort of parameter > options I need to enter or do I need to reinstall the binutils with parameter > options? > On my laptop,

Re: Register allocation issues

2007-09-06 Thread David Edelsohn
> Matt Lee writes: Matt> The problem is, that though the loads can be optimized by pipelining Matt> them. The register allocator has created a dependency by using only r3 Matt> and r4, instead of using the other volatiles. GCC's register allocator currently is designed to minimize the

RE: Register allocation issues

2007-09-06 Thread Dave Korn
On 05 September 2007 23:47, Matt Lee wrote: > Registers r3 to r12 are volatiles. However, for the C code below, > > struct foo { > int a[4]; > } ; > > struct foo p, q; > > void func () > { > memcpy (&p, &q, sizeof (struct foo)); > } > > I am getting a instruction sequence for func() su

Re: Someone has caused regressions in gfortran

2007-09-06 Thread Jan Hubicka
> > The testcase was indeed previously not inlined at all. Shall we add > > -fno-strict-overflow to the testcase then? > > This what I would do, but I am not qualified to make the call. > In addition my working setup is totally broken right now > (at stage1). Could you do the addition to the tes

warnings in GCC source?

2007-09-06 Thread Basile STARYNKEVITCH
Hello All, In most of GCC source code, it is perfectly normal to expect that no warnings should appear, even if the sources are compiled with -Wall or more. Actually the GCC bootstrap process seems to require this. However, I see some occasions where warnings might be quite difficult to avoi

ARM miscompilation of __moddi3 on trunk

2007-09-06 Thread Andrew Haley
This simple test case: #include int ten = 10; int main() { printf ("%lld\n", 92233720368547758LL % ten); return 0; } returns 0 because (afaics) __moddi3 is miscompiled. Breakpoint 4, __moddi3 (u=92233720368547758, v=10) at /home/aph/gcc/trunk/libgcc/../gcc/libgcc2.c:923 (gdb) fin Ru

RE: warnings in GCC source?

2007-09-06 Thread Dave Korn
On 06 September 2007 16:55, Basile STARYNKEVITCH wrote: > 2. generated code: when some C files are generated, it may be hard to > avoid some warnings, typically a generated C function might have unused > arguments (which might be not very easy to detect at generation stage). Generate "__attrib

Re: warnings in GCC source?

2007-09-06 Thread Andreas Schwab
Basile STARYNKEVITCH <[EMAIL PROTECTED]> writes: > So is there an easy way to have some acceptable warnings in GCC? See gcc/Makefile.in, look for -Wno-error. Andreas. -- Andreas Schwab, SuSE Labs, [EMAIL PROTECTED] SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany PGP key fing

Is stmt_ann()->references_memory equal to !ZERO_SSA_OPERANDS (stmt, (SSA_OP_VUSE | SSA_OP_VIRTUAL_DEFS)?

2007-09-06 Thread Richard Guenther
$subject? During early optimization the !ZERO_SSA_OPERANDS check doesn't say the truth (well it does, but only in some sense) while the references_memory flag seems to be updated ok. There is one case, for calls, where we set references_memory unconditionally even if we later might not add any v

Re: Is stmt_ann()->references_memory equal to !ZERO_SSA_OPERANDS (stmt, (SSA_OP_VUSE | SSA_OP_VIRTUAL_DEFS)?

2007-09-06 Thread Diego Novillo
On 9/6/07, Richard Guenther <[EMAIL PROTECTED]> wrote: > $subject? It is not. You can have a statement that references memory and have no virtual operands at all (volatile or totally pruned VOPs for instance). Also, the very first alias analysis pass will find statements with no VOPs but with m

can't reinterpret_cast to/from the same type

2007-09-06 Thread Peter A. Felvegi
hello, i don't know if it's a bug, please clarify: rc.cpp: --8<-- void f() { int x = 0; int y = reinterpret_cast(x); } --8<-- gcc -v: Using built-in specs. Target: i486-linux-gnu Configured with: ../src/configure -v --enable-languages=c,c++,fortran,objc,obj-c++,tre

[lto] project: adding --with-libelf to configure

2007-09-06 Thread Nathan Froyd
The LTO driver requires libelf and currently grovels around in the system directories looking for it, which may not always be the right place to find it. (This bit me when building LTO on our new Linux machines, which do not have libelf installed.) The Right Thing would be to add a --with-libelf

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Andreas Tobler
Kaveh R. GHAZI wrote: (Sorry, first one bounced from gcc@ because it was over 400k) Hi Jan, On sparc-sun-solaris2.10, I'm getting new bootstrap failures in stage2 complaining several times about rtl sharing. I've included four .i files for modules that ICEed during stage2, and the cc1 invocati

RE: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Fu, Chao-Ying
Manuel López-Ibáñez wrote: > > On 05/09/07, Mark Mitchell <[EMAIL PROTECTED]> wrote: > > Summary > > === > > > > We are closing in on Stage 3, previously announced for > September 10th. > > At this point, I'm not aware of any reason to delay that date. Are > > there any Stage 2 patches that

Re: Bootstrap failure under Irix

2007-09-06 Thread Janis Johnson
On Thu, 2007-09-06 at 09:47 +0200, Philippe Schaffnit wrote: > Hi! > > I am seeing the folowing bootstrap failure under SGI Irix: > /USER/philippe/Irix/Gcc_Sources/gcc/config/mips/mips.md: In function > 'gen_fixuns_truncdfsi2': > /USER/philippe/Irix/Gcc_Sources/gcc/config/mips/mips.md:2808: error

Re: Bootstrap broken: ../../gcc/gcc/ada/trans.c:5896: error: too few arguments to function `real_2expN'

2007-09-06 Thread Janis Johnson
On Thu, 2007-09-06 at 15:32 +0200, Christian Joensson wrote: > 2007/9/6, Sandra Loosemore <[EMAIL PROTECTED]>: > > Christian Joensson wrote: > > > Using checkout Thu Sep 6 05:56:16 UTC 2007 (revision 128174), I get a > > > bootstrap failure: > > > > > > ../../gcc/gcc/ada/trans.c: In function `conv

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Jan Hubicka
> (Sorry, first one bounced from gcc@ because it was over 400k) > > Hi Jan, > > On sparc-sun-solaris2.10, I'm getting new bootstrap failures in stage2 > complaining several times about rtl sharing. I've included four .i files > for modules that ICEed during stage2, and the cc1 invocations below.

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Jan Hubicka
> Hi, > I already have fix for this just waiting for Andreas Tobler to verify > that it does what expected. If you could give it a try, it would be > nice. > > The problem is > > /* Called when INSN is being moved from a location near the target of a jump. >We leave a marker of the form (us

RE: GCC 4.3.0 Status Report (2007-09-04)

2007-09-06 Thread Jagasia, Harsha
>On 9/4/07, Mark Mitchell <[EMAIL PROTECTED]> wrote: >> We are closing in on Stage 3, previously announced for September 10th. >> At this point, I'm not aware of any reason to delay that date. Are >> there any Stage 2 patches that people don't think will be submitted by >> that point? >> I still

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Ian Lance Taylor
Jan Hubicka <[EMAIL PROTECTED]> writes: > Producing USE expressions embedding whole INSN. The comment promise > that those will be removed before reorg ends, but they are not. This > patch just adds simple code to remove them in very last dbr_schedule > pass. I see code in dbr_schedule to dele

Re: Register allocation issues

2007-09-06 Thread Matt Lee
On 9/6/07, David Edelsohn <[EMAIL PROTECTED]> wrote: > > Matt Lee writes: > > Matt> The problem is, that though the loads can be optimized by pipelining > Matt> them. The register allocator has created a dependency by using only r3 > Matt> and r4, instead of using the other volatiles. > >

Re: Register allocation issues

2007-09-06 Thread Matt Lee
On 9/6/07, Dave Korn <[EMAIL PROTECTED]> wrote: > On 05 September 2007 23:47, Matt Lee wrote: > > > Registers r3 to r12 are volatiles. However, for the C code below, > > > > struct foo { > > int a[4]; > > } ; > > > > struct foo p, q; > > > > void func () > > { > > memcpy (&p, &q, sizeof (st

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Jan Hubicka
> Jan Hubicka <[EMAIL PROTECTED]> writes: > > > Producing USE expressions embedding whole INSN. The comment promise > > that those will be removed before reorg ends, but they are not. This > > patch just adds simple code to remove them in very last dbr_schedule > > pass. > > I see code in dbr_

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Ian Lance Taylor
Jan Hubicka <[EMAIL PROTECTED]> writes: > > Jan Hubicka <[EMAIL PROTECTED]> writes: > > > > > Producing USE expressions embedding whole INSN. The comment promise > > > that those will be removed before reorg ends, but they are not. This > > > patch just adds simple code to remove them in very l

Re: Register allocation issues

2007-09-06 Thread dje
> Matt Lee writes: Date: Thu, 06 Sep 2007 15:02:52 -0400 From: David Edelsohn <[EMAIL PROTECTED]> Matt> There is no point trying to minimize usage of volatile hard registers, Matt> is there? They are precisely there to be used up as much as needed. Matt> The function is a leaf procedure as we

Re: Register allocation issues

2007-09-06 Thread Segher Boessenkool
load r3, q + 0 load r4, q + 4 store r3, p + 0 store r4, p + 4 load r3, q + 4 load r4, q + 8 store r3, p + 4 store r4, p + 8 These last four lines should be load r3, q + 8 load r4, q + 12 store r3, p + 8 store r4, p + 12 Did you just typo it or do you have a big

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Jan Hubicka
> Jan Hubicka <[EMAIL PROTECTED]> writes: > > > > Jan Hubicka <[EMAIL PROTECTED]> writes: > > > > > > > Producing USE expressions embedding whole INSN. The comment promise > > > > that those will be removed before reorg ends, but they are not. This > > > > patch just adds simple code to remove

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Ian Lance Taylor
Jan Hubicka <[EMAIL PROTECTED]> writes: > The attached patch seems to work on my testcase too. This patch is OK if it passes testing and gets a ChangeLog entry. Thanks. Ian

Re: Register allocation issues

2007-09-06 Thread Matt Lee
On 9/6/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > Matt Lee writes: > Date: Thu, 06 Sep 2007 15:02:52 -0400 > From: David Edelsohn <[EMAIL PROTECTED]> > > Matt> There is no point trying to minimize usage of volatile hard registers, > Matt> is there? They are precisely there to be used

Re: Register allocation issues

2007-09-06 Thread Matt Lee
On 9/6/07, Segher Boessenkool <[EMAIL PROTECTED]> wrote: > > load r3, q + 0 > > load r4, q + 4 > > store r3, p + 0 > > store r4, p + 4 > > load r3, q + 4 > > load r4, q + 8 > > store r3, p + 4 > > store r4, p + 8 > > These last four lines should be > > load r3, q + 8 > load r4, q +

Re: [RFC] missing return warnings

2007-09-06 Thread Mark Mitchell
Jan Hubicka wrote: > For C++ there are number of cases I need to go through dealing with > attempts to not instantiate non-inline methods that won't be needed. I'm not sure exactly what cases you're referring to, but please be careful. For example, changing if/when we instantiate template functi

Re: [RFC] missing return warnings

2007-09-06 Thread Mark Mitchell
Jan Hubicka wrote: > I am basically trying to avoid need for DECL_INLINE (while keeping > DECL_DECLARED_INLINE and DECL_UNINLINABLE for frontend use) - inliner is > now doing all the job himself to figure out what is or isn't needed. That certainly makes sense to me. > With the noreturn warning

Re: [RFC] missing return warnings

2007-09-06 Thread Jan Hubicka
> Jan Hubicka wrote: > > > For C++ there are number of cases I need to go through dealing with > > attempts to not instantiate non-inline methods that won't be needed. > > I'm not sure exactly what cases you're referring to, but please be > careful. For example, changing if/when we instantiate t

Re: [RFC] missing return warnings

2007-09-06 Thread Jan Hubicka
> Jan Hubicka wrote: > > > I am basically trying to avoid need for DECL_INLINE (while keeping > > DECL_DECLARED_INLINE and DECL_UNINLINABLE for frontend use) - inliner is > > now doing all the job himself to figure out what is or isn't needed. > > That certainly makes sense to me. > > > With the

Re: [RFC] missing return warnings

2007-09-06 Thread Mark Mitchell
Jan Hubicka wrote: > One problem that I am concerned about is that C++ frontend is actually > trying to avoid instantiating some stuff that it knows will not be > needed if not inlined. I would kill this logic, but I hope this is the > case where the feature is not worth the maintenance cost. We

GC work status, any comments?

2007-09-06 Thread Laurynas Biveinis
Hello, I've spent last two months at Google quietly working on the boehm-gc branch from my last year's SoC project - http://gcc.gnu.org/wiki/Garbage_collection_tuning . To recap, I've integrated Boehm's GC into GCC proper, adjusted gengtype and GGC support files to generate typed GC allocators for

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Christian Joensson
2007/9/6, Kaveh R. GHAZI <[EMAIL PROTECTED]>: > (Sorry, first one bounced from gcc@ because it was over 400k) > > Hi Jan, > > On sparc-sun-solaris2.10, I'm getting new bootstrap failures in stage2 > complaining several times about rtl sharing. I've included four .i files > for modules that ICEed d

Bootstrap broke today on mipsel-linux.

2007-09-06 Thread David Daney
With revision 128207 Configured: ../trunk/configure --prefix=/home/ddaney/gccsvn/trunk-install --target=mipsel-linux --build=mipsel-linux --host=mipsel-linux --with-gmp=/home/ddaney/mp --with-mpfr=/home/ddaney/mp --with-arch=sb1 --disable-java-awt --without-x --enable-__cxa_atexit --disable-j

Re: RTL sharing bootstrap failure on sparc-sun-solaris2.10

2007-09-06 Thread Kaveh R. GHAZI
On Thu, 6 Sep 2007, Jan Hubicka wrote: > Hi, > I already have fix for this just waiting for Andreas Tobler to verify > that it does what expected. If you could give it a try, it would be > nice. > Honza > > Index: reorg.c > === > ---