--- Comment #71 from hp at gcc dot gnu dot org 2008-08-30 06:27 ---
Created an attachment (id=16169)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16169action=view)
Patch, take 6.
Sigh. This just moves the TREE_STATIC check from the early-return to the
weak-test, since it's wrong
/include --with-mpfr-lib=/usr/lib
--with-gmp-include=/
usr/include --with-gmp-lib=/usr/lib --enable-languages=c++,fortran
--enable-chec
king=release
Thread model: posix
gcc version 4.4.0 20080830 (experimental) [trunk revision 139798] (GCC)
COLLECT_GCC_OPTIONS='-v' '-c' '-O2' '-mtune=generic'
/afs/mpa
--- Comment #1 from martin at mpa-garching dot mpg dot de 2008-08-30 07:17
---
Created an attachment (id=16170)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16170action=view)
a reduced test case
to reproduce to problem, compile with -O2.
--
--- Comment #2 from ajrobb at bigfoot dot com 2008-08-30 07:21 ---
Thanks for that.
I notice that the 64-bit value is now being accumulated in memory rather than
in registers. To be fair, a full 64-bit accumulator need not be maintained -
only the high 32 bits. Could you compile the
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Component|fortran |target
Keywords||ra
--- Comment #7 from mikael dot morin at tele2 dot fr 2008-08-30 11:27
---
I think this should be rejected :
module class_vector
implicit none
type vector
end type vector
end module class_vector
module tools_math
implicit none
interface lin_interp
function
Seemingly, we overlooked some issue when checking the debug patches. Compiling
octopus (tddft.org) with gfortran now gives an ICE:
$ gfortran -g tmp1.f90 tmp.f90
tmp.f90:2: internal compiler error: Segmentation fault
Valgrind shows (when compiling tmp.f90, tmp1.f90 is OK):
==8754== Invalid read
--- Comment #8 from mikael dot morin at tele2 dot fr 2008-08-30 12:12
---
Created an attachment (id=16171)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16171action=view)
another testcase with comments telling how to get/not get the error
Look at the comments in module
--- Comment #9 from mikael dot morin at tele2 dot fr 2008-08-30 12:17
---
Forget comment #8.
I was testing with version 4.3.1
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37274
Compiling the following with -std=c++0x,
auto foo() { }
int main()
{
foo();
}
gives,
test2.cpp: In function âauto foo()â:
test2.cpp:1: internal compiler error: in dependent_type_p, at cp/pt.c:15895
This also gives the same ICE (with -std=c++0x):
void foo(auto i)
{
(void)i;
}
int
--- Comment #1 from chris dot fairles at gmail dot com 2008-08-30 13:14
---
Also, I believe these cases are ice-on-invalid. Jason?
--
chris dot fairles at gmail dot com changed:
What|Removed |Added
--- Comment #4 from sam at gcc dot gnu dot org 2008-08-30 13:45 ---
Subject: Bug 37283
Author: sam
Date: Sat Aug 30 13:44:22 2008
New Revision: 139803
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=139803
Log:
gcc/
PR target/37283
* config/arm/arm.c
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
BugsThisDependsOn||37170
Component|other |middle-end
--- Comment #10 from sfilippone at uniroma2 dot it 2008-08-30 14:00 ---
(In reply to comment #7)
I think this should be rejected :
I think your example should be rejected, but it is not like my code, where
smooth_mesh explicitly USEs class_vector. So, I am not sure what's your point.
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-08-30 14:05 ---
Confirmed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from ubizjak at gmail dot com 2008-08-30 14:05 ---
Confirmed, uninitialized register [st(2)] happens in bb-reorder pass:
before bb-reorder we have:
;; Start of basic block ( 10 3 9 6) - 11
;; lr in7 [sp] 8 [st] 9 [st(1)] 10 [st(2)] 20 [frame]
;; lr use
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-08-30 14:07 ---
Mine. It's only a may_alias pass that ever removes the change_dynamic_type
trees. As this removal fixes up DECL_POINTER_ALIAS_SET this may even cause
wrong code to be emitted during RTL optimization.
--
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-08-30 14:10 ---
What is the ICE? and with what options? I get the same IL on i?86 but no
ICE.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from ubizjak at gmail dot com 2008-08-30 14:16 ---
Adding -fno-reorder-blocks works OK.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37286
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-08-30 14:18 ---
Confirmed. s.file is NULL:
#2 0x0820ff95 in add_src_coords_attributes (die=0xb7c2e428, decl=0xb7cb00d0)
at /home/richard/src/trunk/gcc/dwarf2out.c:12442
12442 add_AT_file (die, DW_AT_decl_file,
--
domob at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |domob at gcc dot gnu dot org
|dot org
--- Comment #10 from rguenther at suse dot de 2008-08-30 14:25 ---
Subject: Re: Trouble with some (C99?) math builtins and
namespace std
On Fri, 29 Aug 2008, paolo dot carlini at oracle dot com wrote:
--- Comment #9 from paolo dot carlini at oracle dot com 2008-08-29 18:46
I am using GCC 4.4.0 20080829 (experimental).
$ cat u.c
void f(unsigned long x);
void g(long x)
{
f((unsigned long)(-1-x)+1);
}
$ gcc -c u.c
u.c: In function 'g':
u.c:4: error: non-trivial conversion at assignment
D.2087
x
D.2087 = -x;
u.c:4: internal compiler error: verify_gimple failed
--- Comment #72 from dave at hiauly1 dot hia dot nrc dot ca 2008-08-30
18:19 ---
Subject: Re: [4.4 Regression]: gcc.dg/weak/weak-1.c
It shouldn't matter for Darwin (which doesn't define ASM_OUTPUT_EXTERNAL,
right?) but will fix the test-case for hppa2.0w-hp-hpux11.11, which BTW
--- Comment #73 from hp at gcc dot gnu dot org 2008-08-30 18:45 ---
(In reply to comment #72)
SUPPORTS_WEAK is defined on hppa2.0w-hp-hpux11.11 depending on GAS
support. See som.h.
Good to know. It can't be compiled cross out of the box though, and the
default is off:
/bin/sh
--- Comment #16 from vmakarov at redhat dot com 2008-08-30 18:52 ---
This is an analysis of the SPEC2006 degradation.
First of all of thanks for reducing the test. I really appreciate
this. Finding it in spec20006 would be very time consuming for me.
The problem looks following
1.
=current- --prefix=/home/regehr --enable-languages=c,c++
--no-create --no-recursion
Thread model: posix
gcc version 4.4.0 20080830 (experimental) (GCC)
[EMAIL PROTECTED]:~/volatile/tmp14$ cat small.c
typedef int int32_t;
typedef unsigned char uint8_t;
typedef unsigned int uint32_t;
static inline
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jason at gcc dot gnu dot org
|dot org
--- Comment #2 from jakub at gcc dot gnu dot org 2008-08-30 19:34 ---
Created an attachment (id=16172)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16172action=view)
gcc44-pr37287.patch
Patch I'm going to bootstrap/regtest.
For DW_TAG_module with DW_AT_declaration we really
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-08-30 22:17 ---
../../src/bfd/xcofflink.c: In function '_bfd_xcoff_canonicalize_dynamic_reloc':
../../src/bfd/xcofflink.c:399: internal compiler error: in purge_dead_edges, at
cfgrtl.c:2327
Please submit a full bug report,
with
--- Comment #3 from kkojima at gcc dot gnu dot org 2008-08-30 22:24 ---
Subject: Bug 37270
Author: kkojima
Date: Sat Aug 30 22:23:20 2008
New Revision: 139809
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=139809
Log:
PR target/37270
* config/sh/sh.h
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-08-30 22:25 ---
Confirmed on powerpc-linux-gnu also.
-- Pinski
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pinskia at gcc dot gnu dot org 2008-08-30 22:27 ---
One more thing I forgot to mention is that the ICE is during expand time.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37285
--- Comment #4 from pinskia at gcc dot gnu dot org 2008-08-30 23:02 ---
Here is a C++ testcase (you can convert it to C if you want by changing the
references to pointers and such):
int gni97 = 97, gnj97 = 33;
double gnu[97];
extern void g(void);
double f(double min, float max)
{
--- Comment #5 from pinskia at gcc dot gnu dot org 2008-08-30 23:12 ---
This used to work before IRA (4.4.0 20080824).
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from jason at gcc dot gnu dot org 2008-08-30 23:14 ---
Subject: Bug 37288
Author: jason
Date: Sat Aug 30 23:12:45 2008
New Revision: 139811
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=139811
Log:
PR c++/37288
* pt.c (dependent_type_p): Don't
We build a large amount of software using a src tree and a separate object
tree.
We have recently upgraded from gcc v3.4 to gcc v4.3
$ gcc --version
gcc (GCC) 4.3.2
Copyright (C) 2008 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-08-30 23:31 ---
*** This bug has been marked as a duplicate of 19541 ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #12 from pinskia at gcc dot gnu dot org 2008-08-30 23:31
---
*** Bug 37291 has been marked as a duplicate of this bug. ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
We build a large amount of software using a src tree and a separate object
tree.
We have recently upgraded from gcc v3.4 to gcc v4.3
$ gcc --version
gcc (GCC) 4.3.2
Copyright (C) 2008 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
Revision 139762
Jump to revision:
Author: hubicka
Date: Fri Aug 29 11:39:04 2008 UTC (35 hours, 39 minutes ago)
Log Message:
* doc/invoke.texi (-fipa-cp): Enabled by default at -O2/-Os/-O3
(-fipa-cp-clone): Enabled by default at -O3.
* opts.c
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-08-30 23:37 ---
*** This bug has been marked as a duplicate of 19541 ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #13 from pinskia at gcc dot gnu dot org 2008-08-30 23:37
---
*** Bug 37292 has been marked as a duplicate of this bug. ***
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19541
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
CC||pinskia at gcc dot gnu dot
|
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2008-08-31 00:05
---
Subject: Bug 36895
Author: jvdelisle
Date: Sun Aug 31 00:04:33 2008
New Revision: 139813
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=139813
Log:
2008-08-30 Jerry DeLisle [EMAIL PROTECTED]
PR
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2008-08-31 00:09
---
Subject: Bug 36895
Author: jvdelisle
Date: Sun Aug 31 00:07:58 2008
New Revision: 139814
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=139814
Log:
2008-08-30 Jerry DeLisle [EMAIL PROTECTED]
PR
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2008-08-31 00:13
---
Fixed on 4.4 for scalar character internal unit.
--
jvdelisle at gcc dot gnu dot org changed:
What|Removed |Added
The following illustrates this:
character(30) :: line(3)
namelist /stuff/ n
n = 123
line =
write(line,nml=stuff)
write(*,*) 1),line(1)
write(*,*) 2),line(2)
write(*,*) 3),line(3)
end
$ ./a.out
1)STUFF N=123, /
2)
3)
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2008-08-31 00:24
---
Correction. -std=f95 disallows it.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37294
--- Comment #4 from kkojima at gcc dot gnu dot org 2008-08-31 00:28 ---
Fixed.
--
kkojima at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-08-31 00:52 ---
So we end up creating the _T.303 decl and marking it as DECL_WEAK.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-08-31 00:54 ---
Reducing a testcase right now with version 4.4.0 20080824.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37293
--- Comment #3 from pinskia at gcc dot gnu dot org 2008-08-31 01:57 ---
Here is a short reduced testcase:
void f(int);
inline void _M_reset (int *a) throw()
{
f (*a);
}
bool _M_finished();
int _M_curr_index;
void _M_allocate_single_object()
{
if (__builtin_expect (_M_finished(),
--- Comment #4 from pinskia at gcc dot gnu dot org 2008-08-31 02:00 ---
Simple patch which fixes the DECL_WEAK issue (I think).
Index: cgraphunit.c
===
--- cgraphunit.c(revision 139811)
+++ cgraphunit.c
--- Comment #3 from paolo dot carlini at oracle dot com 2008-08-31 02:14
---
Manuel, if you agree I would quickly handle this, by just adding a
/* { dg-message note: expected 'signed char *' but argument is of type
'unsigned char *' } */
at line 5. Ok?
--
paolo dot carlini at
--- Comment #5 from pinskia at gcc dot gnu dot org 2008-08-31 02:24 ---
That patch fixes the reduced testcase, I cannot do a bootstrap/test on it
today.
Jack can you do one for me with the patch?
--
pinskia at gcc dot gnu dot org changed:
What|Removed
--- Comment #6 from howarth at nitro dot med dot uc dot edu 2008-08-31
02:39 ---
Starting a bootstrap now.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37293
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-08-31 02:43 ---
This was fixed by:
http://gcc.gnu.org/ml/gcc-patches/2008-06/msg00654.html
I will go ahead and add a testcase later this week.
--
pinskia at gcc dot gnu dot org changed:
What|Removed
--- Comment #8 from pinskia at gcc dot gnu dot org 2008-08-31 02:45 ---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #74 from dave at hiauly1 dot hia dot nrc dot ca 2008-08-31
03:19 ---
Subject: Re: [4.4 Regression]: gcc.dg/weak/weak-1.c
--- Comment #71 from hp at gcc dot gnu dot org 2008-08-30 06:27 ---
Created an attachment (id=16169)
--
configure fails because there are too many arguments in the test
statement. I believe the following change needs to be back ported:
2007-08-23 Brian Sidebotham [EMAIL PROTECTED]
* configure.ac (leb128): Modify sed statement to work with any
binutils version string.
*
--- Comment #7 from howarth at nitro dot med dot uc dot edu 2008-08-31
04:18 ---
The proposed patch changes the previous compilation failure into...
/bin/sh ../libtool --tag CXX --mode=compile
/sw/src/fink.build/gcc44-4.3.999-20080830/darwin_objdir/./gcc/xgcc
-shared-libgcc -B/sw/src
what I can do about this issue. Mostly we need to look at
where we change from weak to non weak and then fix up those call
statements. Honza, does that seem correct?
Thanks,
Andrew Pinski
/bin/sh ../libtool --tag CXX --mode=compile
/sw/src/fink.build/gcc44-4.3.999-20080830/darwin_objdir
that seem correct?
Thanks,
Andrew Pinski
/bin/sh ../libtool --tag CXX --mode=compile
/sw/src/fink.build/gcc44-4.3.999-20080830/darwin_objdir/./gcc/xgcc
-shared-libgcc -B/sw/src/fink.build/gcc44-4.3.999-20080830/
darwin_objdir/./gcc
-nostdinc++
-L/sw/src/fink.build/gcc44-4.3.999-20080830
--- Comment #9 from howarth at nitro dot med dot uc dot edu 2008-08-31
04:25 ---
Created an attachment (id=16173)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16173action=view)
preprocessed source for bitmap_allocator.cc with proposed patch
--
--- Comment #10 from howarth at nitro dot med dot uc dot edu 2008-08-31
04:26 ---
Created an attachment (id=16174)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16174action=view)
assembly file for bitmap_allocator.cc with proposed patch
--
Recent bootstraps on i386-unknown-freebsd8.0 lead to
gmake[4]: Leaving directory
`/usr/home/kargl/gcc/obj/i386-unknown-freebsd8.0/libgcc'
/usr/home/kargl/gcc/obj/./gcc/xgcc -B/usr/home/kargl/gcc/obj/./gcc/
-B/usr/home/kargl/work/i386-unknown-freebsd8.0/bin/
--
kargl at gcc dot gnu dot org changed:
What|Removed |Added
Severity|normal |critical
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37296
68 matches
Mail list logo
