--- Comment #1 from pinskia at gcc dot gnu dot org 2006-11-26 08:01 ---
This comes down to a tree combiner really.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from pinskia at gcc dot gnu dot org 2006-11-26 08:05 ---
Here is a short testcase:
int f(int a, int b)
{
int i;
int c = a=b?a:b;
for(i = 0;i1000;i++) ; // Needed otherwise Jump threading gets in the way
int d = c=a?c:a;
return d;
}
--
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-11-26 08:15 ---
Actually I think it is better to mark this as a dup of bug 14541 which is all
about combining expressions for math builtins.
*** This bug has been marked as a duplicate of 14541 ***
--
pinskia at gcc dot gnu
--- Comment #14 from pinskia at gcc dot gnu dot org 2006-11-26 08:15
---
*** Bug 29985 has been marked as a duplicate of this bug. ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #17 from paolo dot bonzini at lu dot unisi dot ch 2006-11-26
09:05 ---
Subject: Re: [4.3 Regression] build/genconditions
../../gcc/gcc/config/pa/pa.md tmp-condmd.c: /bin/sh: 13354 Memory
fault(coredump)
I wonder if it is enough to just add DF_HARD_REGS in the df_init
All the testsuite failures (except libgomp.c++/ctor-9.C) with Sun as/ld
http://gcc.gnu.org/ml/gcc-testresults/2006-11/msg00764.html
are of the form:
ld: fatal: relocation error: R_SPARC_TLS_LE_HIX22: file /var/tmp//ccuoqyTO.o:
symbol unknown: bad symbol type SECT: symbol type must be TLS
--- Comment #18 from stevenb dot gcc at gmail dot com 2006-11-26 09:19
---
Subject: Re: [4.3 Regression] build/genconditions
../../gcc/gcc/config/pa/pa.md tmp-condmd.c: /bin/sh: 13354 Memory
fault(coredump)
Just adding DF_HARD_REGS is not enough. At least this bit:
- if (use)
This failure with Sun as/ld
http://gcc.gnu.org/ml/gcc-testresults/2006-11/msg00764.html
is of the form:
ld: fatal: relocation error: R_SPARC_HI22: file /var/tmp//cc8eBELF.o: symbol
base: relocation illegal for TLS symbol
It's a bug in Sun as, it doesn't switch section between .tbss and .bss:
--- Comment #9 from razya at gcc dot gnu dot org 2006-11-26 10:00 ---
Subject: Bug 29122
Author: razya
Date: Sun Nov 26 10:00:18 2006
New Revision: 119220
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119220
Log:
2006-11-26 Razya Ladklesky [EMAIL PROTECTED]
PR
--- Comment #2 from paolo at gcc dot gnu dot org 2006-11-26 10:04 ---
Subject: Bug 29385
Author: paolo
Date: Sun Nov 26 10:04:25 2006
New Revision: 119221
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119221
Log:
2006-11-26 Paolo Carlini [EMAIL PROTECTED]
PR
--- Comment #3 from pcarlini at suse dot de 2006-11-26 10:05 ---
Done.
--
pcarlini at suse dot de changed:
What|Removed |Added
Status|NEW
From Ion again:
http://gcc.gnu.org/ml/libstdc++/2006-11/msg00114.html
--
Summary: More stl_tree.h enhancements: improving operator=
Product: gcc
Version: 4.3.0
Status: UNCONFIRMED
Severity: enhancement
Priority: P3
--
pcarlini at suse dot de changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2006-11-26 12:26
---
Subject: Bug 29892
Author: fxcoudert
Date: Sun Nov 26 12:25:50 2006
New Revision: 119223
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119223
Log:
PR fortran/29892
* trans-intrinsic.c
testcase:
#define max(a,b) quirk
#include limits
output:
/usr/include/c++/4.1.2/limits:293:22: error: macro max requires 2 arguments,
but only 1 given
/usr/include/c++/4.1.2/limits:323:23: error: macro max requires 2 arguments,
but only 1 given
/usr/include/c++/4.1.2/limits:377:23: error: macro
--- Comment #14 from vapier at gentoo dot org 2006-11-26 12:33 ---
sorry for the delay ... i just updated my local tree and proposed patch works
great, thanks :)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29114
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2006-11-26 12:34
---
Fixed for array bounds checking on mainline. I'll backport the patch to 4.2 in
some time.
We still need to figure something out for the substrings case (see comment #0):
Secondly, gfc_conv_substring_expr
--- Comment #11 from chris_clayton at f1internet dot com 2006-11-26 12:54
---
I'm the reporter of 29809, which is a duplicate of this bug. I'm not familiar
with the workflow here, so do I simply wait for 4.1.2 or is there likely to be
patch to test before 4.1.2 is released? Is the
--- Comment #12 from rguenth at gcc dot gnu dot org 2006-11-26 13:09
---
Usually you wait until a patch is available or 4.1.2 is released (supposed the
problem is fixed there). You can try testing the ad-hoc fix in PR29809 comment
#4.
--
--- Comment #1 from pcarlini at suse dot de 2006-11-26 13:38 ---
As a matter of fact, I think that the undefs in stl_algobase.h are legacy
stuff, and should go away. In my opinion, we should always error in such cases,
because nobody knows whether the user means his own max / min macro,
--- Comment #1 from manu at gcc dot gnu dot org 2006-11-26 13:38 ---
Hi, Richard.
I don't understand completely what you mean. Is the problem in the builtin
exp() ? Should it check its argument for large numbers?
--
manu at gcc dot gnu dot org changed:
What
--- Comment #2 from rguenth at gcc dot gnu dot org 2006-11-26 13:56 ---
The problem is that we believe we can handle all errno checking/setting via
the expand_errno_check() routine which is not true for overflow/underflow but
only for invalid arguments that result in a NaN.
--
--- Comment #4 from manu at gcc dot gnu dot org 2006-11-26 13:57 ---
This will be fixed by the new Wconversion option. Actually, it has been fixed
already for gcc version 4.3.0 20061124 (experimental).
[EMAIL PROTECTED]:~$ local/bin/g++ -c -Wconversion pr18821.C
pr18821.C: In function
--- Comment #5 from manu at gcc dot gnu dot org 2006-11-26 13:59 ---
For reference:
Author: manu
Date: Fri Nov 24 01:50:33 2006
New Revision: 119143
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119143
Log:
2006-11-24 Manuel Lopez-Ibanez [EMAIL PROTECTED]
PR c/2707
--- Comment #1 from ghazi at gcc dot gnu dot org 2006-11-26 14:02 ---
Is this a known bug or do we need to report it to Sun?
If known, is there a patch we can recommend in the Solaris-specific
installation docs?
--
ghazi at gcc dot gnu dot org changed:
What|Removed
--- Comment #1 from ghazi at gcc dot gnu dot org 2006-11-26 14:03 ---
Is this a known bug or do we need to report it to Sun?
If known, is there a patch we can recommend in the Solaris-specific
installation docs?
--
ghazi at gcc dot gnu dot org changed:
What|Removed
--
manu at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18821
--- Comment #13 from patchapp at dberlin dot org 2006-11-26 14:30 ---
Subject: Bug number PR25620
A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2006-11/msg01758.html
--
--- Comment #2 from ebotcazou at gcc dot gnu dot org 2006-11-26 14:35
---
Is this a known bug or do we need to report it to Sun?
I quickly skimmed through Sun linker patches but didn't find anything.
The problem is of course that all works flawlessly with the Sun toolchain
in default
--- Comment #2 from ebotcazou at gcc dot gnu dot org 2006-11-26 14:38
---
Is this a known bug or do we need to report it to Sun?
Since the Sun toolchain doesn't use the assembler by default, it is very
likely unknown to them.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29987
--- Comment #3 from pinskia at gcc dot gnu dot org 2006-11-26 18:08 ---
Subject: Bug 29982
Author: pinskia
Date: Sun Nov 26 18:08:14 2006
New Revision: 119227
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119227
Log:
2006-11-25 Andrew Pinski [EMAIL PROTECTED]
PR
--- Comment #10 from pinskia at gcc dot gnu dot org 2006-11-26 18:15
---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #14 from pinskia at gcc dot gnu dot org 2006-11-26 18:25
---
The problem with RTH's patch was the use of t which shadowed the variable in
the function which is used for setting the locus. I am testing the corrected
patch now.
--
--- Comment #2 from jakub at gcc dot gnu dot org 2006-11-26 19:16 ---
That's actually ICE on invalid, you can't throw out of a parallel region.
OpenMP 2.5, section 2.4 says:
A throw executed inside a parallel region must cause execution to resume
within the same parallel region, and it
--- Comment #4 from manu at gcc dot gnu dot org 2006-11-26 19:27 ---
(In reply to comment #3)
(In reply to comment #2)
The problem is that we reset TREE_OVERFLOW:
It would seem it overflows incremented, and underflow's decremented,
only a terminal non-zero count would represent
--- Comment #5 from pinskia at gcc dot gnu dot org 2006-11-26 19:51 ---
I have a fix by not never producing MAX_EXPRa,a or MIN_EXPRa,a in reassoc.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from membar at gcc dot gnu dot org 2006-11-26 19:53 ---
Subject: Bug 29151
Author: membar
Date: Sun Nov 26 19:53:11 2006
New Revision: 119230
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119230
Log:
PR libgcj/29151:
* java/lang/natWin32Process.cc (startProcess):
--- Comment #6 from pinskia at gcc dot gnu dot org 2006-11-26 20:03 ---
Here is the patch which I am testing:
Index: tree-ssa-reassoc.c
===
--- tree-ssa-reassoc.c (revision 119229)
+++ tree-ssa-reassoc.c (working copy)
@@
--- Comment #22 from tkoenig at gcc dot gnu dot org 2006-11-26 20:39
---
Created an attachment (id=12696)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12696action=view)
Serious attempt
Hi folks,
here is a serious attempt at the patch. Jerry, if
you could give it a spin, I'd be
--- Comment #23 from tkoenig at gcc dot gnu dot org 2006-11-26 20:43
---
Created an attachment (id=12697)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12697action=view)
Test case
Here's a test case for reading and writing with
a restricted subrecord length.
--
--- Comment #13 from chris_clayton at f1internet dot com 2006-11-26 21:22
---
(In reply to comment #12)
Usually you wait until a patch is available or 4.1.2 is released (supposed the
problem is fixed there). You can try testing the ad-hoc fix in PR29809
comment
#4.
OK, I've
--- Comment #24 from jvdelisle at verizon dot net 2006-11-26 22:27 ---
Subject: Re: implement unformatted files with subrecords
(Intel style)
I have reviewed the patch and I have one minor comment. I suggest that the
continued flag be placed in with the unit flags similar to
--- Comment #19 from dave at hiauly1 dot hia dot nrc dot ca 2006-11-27
00:28 ---
Subject: Re: [4.3 Regression] build/genconditions
../../gcc/gcc/config/pa/pa.md tmp-condmd.c: /bin/sh: 13354 MO
Subject: Re: [4.3 Regression] build/genconditions
../../gcc/gcc/config/pa/pa.md
--- Comment #6 from bangerth at dealii dot org 2006-11-27 02:00 ---
Excellent, this is exactly the quote that settles this. For
reference, 14.3.1/3 comes with a (as usual non-normative) example:
[Example:
templateclass T struct A {
static T t;
};
typedef
--- Comment #10 from pinskia at gcc dot gnu dot org 2006-11-27 02:58
---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from mmitchel at gcc dot gnu dot org 2006-11-27 03:39
---
Subject: Bug 29886
Author: mmitchel
Date: Mon Nov 27 03:38:57 2006
New Revision: 119242
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119242
Log:
PR c++/29886
* parser.c (cp_parser): Add
--- Comment #3 from mmitchel at gcc dot gnu dot org 2006-11-27 03:46
---
Fixed in 4.3.0.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
Configured with: ../gcc-4.1.1/configure --prefix=/opt/local
--enable-languages=c,c++,java,objc,obj-c++,fortran
--libdir=/opt/local/lib/gcc41 --includedir=/opt/local/include/gcc41
--infodir=/opt/local/share/info --mandir=/opt/local/share/man
--with-local-prefix=/opt/local --with-system-zlib
--- Comment #1 from yves at gnu-darwin dot org 2006-11-27 04:30 ---
Created an attachment (id=12699)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12699action=view)
NSFramework_PopplerKit.mi
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29990
--- Comment #4 from mmitchel at gcc dot gnu dot org 2006-11-27 04:38
---
Subject: Bug 29886
Author: mmitchel
Date: Mon Nov 27 04:38:34 2006
New Revision: 119243
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119243
Log:
PR c++/29886
* parser.c (cp_parser): Add
--- Comment #5 from mmitchel at gcc dot gnu dot org 2006-11-27 04:39
---
Fixed in 4.1.2.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-11-27 04:49 ---
I don't think you can use -flat_namespace with dynamic libraries and libstdc++.
Also this should have been fixed by:
* config/darwin.c (machopic_select_section): Remove the hack to
mark ::operator
On Mon, 2006-11-27 at 04:49 +, pinskia at gcc dot gnu dot org wrote:
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-11-27 04:49
---
I don't think you can use -flat_namespace with dynamic libraries and
libstdc++.
Also I don't think this is a GCC issue. I think it is
--- Comment #3 from pinskia at gmail dot com 2006-11-27 04:51 ---
Subject: Re: Linking fails because __ZdlPv can't be a
weak definition
On Mon, 2006-11-27 at 04:49 +, pinskia at gcc dot gnu dot org wrote:
--- Comment #2 from pinskia at gcc dot gnu dot org
--- Comment #6 from mmitchel at gcc dot gnu dot org 2006-11-27 04:56
---
Subject: Bug 29886
Author: mmitchel
Date: Mon Nov 27 04:56:02 2006
New Revision: 119244
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119244
Log:
PR c++/29886
* parser.c (cp_parser): Add
--- Comment #7 from mmitchel at gcc dot gnu dot org 2006-11-27 04:56
---
Fixed in 4.2.0.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from pinskia at gcc dot gnu dot org 2006-11-27 05:01 ---
Subject: Bug 29982
Author: pinskia
Date: Mon Nov 27 05:00:58 2006
New Revision: 119245
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119245
Log:
2006-11-26 Andrew Pinski [EMAIL PROTECTED]
PR
--- Comment #5 from pinskia at gcc dot gnu dot org 2006-11-27 05:01 ---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #15 from pinskia at gcc dot gnu dot org 2006-11-27 05:46
---
(In reply to comment #14)
The problem with RTH's patch was the use of t which shadowed the variable in
the function which is used for setting the locus. I am testing the corrected
patch now.
That fixed most
On Mon, 2006-11-27 at 05:46 +, pinskia at gcc dot gnu dot org wrote:
That fixed most of the failures but there are still some ICEs that need to be
fixed.
I have a fix for those ICEs, it is just checking for DECL_P.
-- Pinski
--- Comment #16 from pinskia at gmail dot com 2006-11-27 05:51 ---
Subject: Re: [tree-ssa] badly formatted warnings for SRA
replacements used uninitialized
On Mon, 2006-11-27 at 05:46 +, pinskia at gcc dot gnu dot org wrote:
That fixed most of the failures but there are
--- Comment #2 from paulthomas2 at wanadoo dot fr 2006-11-27 06:19 ---
Subject: Re: [4.2/4.3 regression] ICE on optional arg
Andrew,
We are calling build_int_cst on a real type.
Yes, I discovered the same on my flight back to France last night. Will
come up with a fix tonight.
--- Comment #17 from pinskia at gcc dot gnu dot org 2006-11-27 06:41
---
Here is the patch which passes the C++ testsuite, I have to do a full
bootstrap/testsuite run still but I am happy with it currrently which is why I
am pasting it here:
Index: error.c
--- Comment #20 from bonzini at gnu dot org 2006-11-27 07:16 ---
This may be a df bug too. I don't know if it is ok to expect, when
DF_HARD_REGS is set, that the list of defs include a def for every hard
register that is call-clobbered and live at the call?
--
bonzini at gnu dot
--- Comment #10 from pinskia at gcc dot gnu dot org 2006-11-27 07:51
---
(In reply to comment #9)
Though the good question is why we have those PHIs still there even though
they
don't do anything except produce a copy. All the copyprop passes as far as I
can tell skip them
65 matches
Mail list logo
