--- Comment #26 from pinskia at gcc dot gnu dot org 2008-04-30 04:13
---
A simple fix for the patch is to add:
&& TREE_CODE (TREE_TYPE (TREE_OPERAND (def_rhs, 0))) != ARRAY_TYPE
But note, we still get worse code with the C front-end than with the C++
front-end but that is due to
--- Comment #25 from pinskia at gcc dot gnu dot org 2008-04-30 04:04
---
(In reply to comment #24)
> This is the patch which I am testing for the testcase in comment #11 as
> mentioned we don't use NOP_EXPR but instead VIEW_CONVERT_EXPR as
> VIEW_CONVERT_EXPR can work on non gimple regi
--- Comment #24 from pinskia at gcc dot gnu dot org 2008-04-30 03:58
---
Created an attachment (id=15551)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15551&action=view)
Patch for comment #11 on the tree level
This is the patch which I am testing for the testcase in comment #11
--- Comment #23 from pinskia at gcc dot gnu dot org 2008-04-30 03:47
---
(In reply to comment #21)
> (In reply to comment #20)
>
> > Here are 2 alternatives I've been playing with:
>
> Any news on this patch?
The second patch looks like what is was done on the RTL level with respect
--- Comment #2 from victork at gcc dot gnu dot org 2008-04-30 02:48 ---
Hello,
I've tried to complile the attached kernel_build.ii on my SUSE SLES 10 x86_64
machine, but got a bunch of compile errors like this:
In file included from /usr/lib/gcc/i486-linux-gnu/4.3.1/include/xmmintrin.h
--- Comment #6 from bdavis at gcc dot gnu dot org 2008-04-30 00:55 ---
notice the 6 minutes between the above posts :)
jerry was correct, it needed a NULL check.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35940
--- Comment #5 from bdavis at gcc dot gnu dot org 2008-04-30 00:54 ---
close. here is a better patch.
http://gcc.gnu.org/ml/fortran/2008-04/msg00278.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35940
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2008-04-30 00:00
---
Note the last line of patch is checking for b != NULL before using it.
Probably want to do likewise for the added test condition above.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35940
--- Comment #3 from sam at gcc dot gnu dot org 2008-04-29 21:46 ---
This bug is now fixed in GCC 4.4.0.
--
sam at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from sam at gcc dot gnu dot org 2008-04-29 21:44 ---
Subject: Bug 35792
Author: sam
Date: Tue Apr 29 21:43:39 2008
New Revision: 134810
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134810
Log:
2008-04-29 Ed Schonberg <[EMAIL PROTECTED]>
gcc/ada/
P
--- Comment #2 from kas at fi dot muni dot cz 2008-04-29 21:39 ---
Interesting. When rewritten to use union, it works even with -O3. Thanks!
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36082
--- Comment #8 from bkoz at gcc dot gnu dot org 2008-04-29 21:37 ---
Fixed on trunk and gcc-4_3-branch
--
bkoz at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #127 from pinskia at gcc dot gnu dot org 2008-04-29 21:24
---
*** Bug 36082 has been marked as a duplicate of this bug. ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-04-29 21:24 ---
It is more than unportable, it contains undefined code as you violate C/C++
aliasing rules:
d = *(unsigned long *)e1;
*(unsigned long *)e1 = *(unsigned long *)e2;
*(unsigned long *)e2 = d;
Y
In my program I use recursive quick-sort, which is apparently miscompiled by my
gcc (Fedora 8, x86_64), when I use -O3 or -O2 -finline-functions. With these
options, the data get out of the qsort routine only partially sorted. With -O2
it is OK, and when I add a debugging printf() at the end of the
--- Comment #2 from mrs at mythic-beasts dot com 2008-04-29 21:02 ---
Thanks for the explanation. I have filed a bug on glibc with a simple
workaround (http://sources.redhat.com/bugzilla/show_bug.cgi?id=6428). Changing
binutils is not necessary to fix glibc's configure check.
--
mr
--- Comment #7 from bkoz at gcc dot gnu dot org 2008-04-29 19:45 ---
Subject: Bug 35922
Author: bkoz
Date: Tue Apr 29 19:44:18 2008
New Revision: 134806
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134806
Log:
2008-04-29 Benjamin Kosnik <[EMAIL PROTECTED]>
PR libstd
--- Comment #3 from bdavis at gcc dot gnu dot org 2008-04-29 19:23 ---
this looks promising:
Index: gcc/gcc/fortran/simplify.c
===
--- gcc/gcc/fortran/simplify.c (revision 134801)
+++ gcc/gcc/fortran/simplify.c (working c
--- Comment #8 from pinskia at gcc dot gnu dot org 2008-04-29 19:11 ---
>And for syncing the threads, that's the easiest way.
Use mutexes then, they are designed exactly for this.
>I don't want so serialize the threads, I want to use all 2 processors of my
dual-core.
You are serializi
--- Comment #7 from snes2002 at freenet dot de 2008-04-29 19:06 ---
Subject: Re: gcc optimizations and threads (pthread)
thanks for the quick reply.
But this code works without opimizations. Even with complex constructs.
For the solution of my problem i can't use lock's.
I don't want
--- Comment #6 from hjl dot tools at gmail dot com 2008-04-29 19:03 ---
Revision 134799 works.
*** This bug has been marked as a duplicate of 36078 ***
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
--- Comment #5 from hjl dot tools at gmail dot com 2008-04-29 19:03 ---
*** Bug 36074 has been marked as a duplicate of this bug. ***
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36078
--- Comment #6 from snes2002 at freenet dot de 2008-04-29 18:59 ---
This c-code works with optimizations turned off.
With borland compiler it also works with optimizations turned on.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36081
--- Comment #5 from snes2002 at freenet dot de 2008-04-29 18:57 ---
Created an attachment (id=15550)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15550&action=view)
assembler listing
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36081
--- Comment #4 from snes2002 at freenet dot de 2008-04-29 18:56 ---
Created an attachment (id=15549)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15549&action=view)
preprocessed file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36081
--- Comment #3 from snes2002 at freenet dot de 2008-04-29 18:55 ---
Created an attachment (id=15548)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15548&action=view)
c source
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36081
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-04-29 18:54 ---
This is correct behavior, you need either use volatile or use locks.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from snes2002 at freenet dot de 2008-04-29 18:54 ---
Created an attachment (id=15547)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15547&action=view)
gcc output
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36081
gcc version 4.2.1 (Debian 4.2.1-3)
System: debian-installation from knoppix 5.1.1
When optimizations are turned on (-O, -O1, -O2, -O3),
two or more threads don't recognize when the value of a global variable
changes.
It doesn't matter if it's a global variable or a pointer to a local variable.
The
--- Comment #8 from hjl dot tools at gmail dot com 2008-04-29 18:44 ---
On IA32, all 8 byte types should be aligned at 4 byte and 16byte types
should be aligned at 16 byte. The only discrepancy is alignment of
8byte DFP type, _Decimal64. Gcc aligns it at 8 byte. But we should
treat its a
--
ubizjak at gmail dot com changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last reconfirmed|0
--- Comment #7 from ubizjak at gmail dot com 2008-04-29 18:06 ---
The alignment of TDmode is not yet decided, see
http://groups.google.com/group/ia32-abi/browse_thread/thread/fc6db344b62dea76
Reopened as P1 bug, blocking 4.3.1 release, as per suggestion in
http://gcc.gnu.org/ml/gcc-patc
--- Comment #5 from pinskia at gcc dot gnu dot org 2008-04-29 17:09 ---
*** Bug 36080 has been marked as a duplicate of this bug. ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-04-29 17:09 ---
*** This bug has been marked as a duplicate of 33661 ***
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--
Hi,
when doing 'register int val asm("rx")' in a template function the explicit
register allocation is ignored and val is given an arbitrary register without
any notice. The following test shows that the asm-statement is ignored.
class X {
public:
template int func_template(int *x);
int no_t
--- Comment #6 from dave at hiauly1 dot hia dot nrc dot ca 2008-04-29
16:08 ---
Subject: Re: [4.4 Regression] FAIL: gcc.c-torture/execute/20040709-1.c
execution at -O2 and above
> I first saw this in 133010.
It was introduced in 132969.
Dave
--
http://gcc.gnu.org/bugzilla/show_
--
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
--- Comment #10 from rguenth at gcc dot gnu dot org 2008-04-29 16:03
---
The fold missed optimizations are fixed.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15255
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-04-29 16:02 ---
Subject: Bug 36078
Author: rguenth
Date: Tue Apr 29 16:01:36 2008
New Revision: 134799
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134799
Log:
2008-04-29 Richard Guenther <[EMAIL PROTECTED]>
PR
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-04-29 16:02 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #9 from rguenth at gcc dot gnu dot org 2008-04-29 16:00 ---
Subject: Bug 15255
Author: rguenth
Date: Tue Apr 29 15:59:43 2008
New Revision: 134798
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134798
Log:
2008-04-29 Richard Guenther <[EMAIL PROTECTED]>
PR
--- Comment #20 from hubicka at gcc dot gnu dot org 2008-04-29 15:40
---
Created an attachment (id=15546)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15546&action=view)
New patch
Hi,
this patch implements idea of having temporary read-only register with
REG_EQUIV note that will
--- Comment #8 from rguenth at gcc dot gnu dot org 2008-04-29 15:13 ---
Also fixed on the 4.3 branch. Unassigning.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #7 from rguenth at gcc dot gnu dot org 2008-04-29 15:13 ---
Subject: Bug 36077
Author: rguenth
Date: Tue Apr 29 15:12:21 2008
New Revision: 134794
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134794
Log:
2008-04-29 Richard Guenther <[EMAIL PROTECTED]>
PR
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-04-29 14:46 ---
ISVs usually build with compilers from the stone-age to be able to run their
applications on all still supported OS versions. So it is very unlikely that
they will hit this problem. In fact if we enable this workar
--- Comment #2 from dominiq at lps dot ens dot fr 2008-04-29 14:44 ---
If this may help, the test case compiles on powerpc-apple-darwin9, revision
134791+patches, but fails on i686-apple-darwin9 revision 134751+patches.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36078
--- Comment #8 from rguenth at gcc dot gnu dot org 2008-04-29 14:17 ---
We already handle most of the foldings in fold_plusminus_mult_expr, just the
A + A -> 2 * A folding is not done (for a reason).
We also miss (A + A) * Cst -> A * 2 * Cst, which is what I am going to
implement
in add
--- Comment #6 from rguenth at gcc dot gnu dot org 2008-04-29 13:53 ---
Subject: Bug 36077
Author: rguenth
Date: Tue Apr 29 13:52:53 2008
New Revision: 134792
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134792
Log:
2008-04-29 Richard Guenther <[EMAIL PROTECTED]>
PR
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-04-29 13:53 ---
Fixed on the trunk.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Known to w
--- Comment #2 from cnstar9988 at gmail dot com 2008-04-29 13:36 ---
This PR 36079 can't appear on buglist.
target_milestone=4.3.1
short_desc=4.3
?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36079
--- Comment #11 from ilmarw at simula dot no 2008-04-29 13:36 ---
Fixed in version 4.3.0.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28482
--- Comment #1 from ubizjak at gmail dot com 2008-04-29 13:15 ---
Since there are many workarounds for this problem I think that this problem
should be fixed elsewhere by either (a) fixing the kernel or (b) fixing the
application that is affected by the problem
a) A two-liner patch to t
This bug is opened to track the issue with cld insn.
gcc-4.3.0 does not emit cld instruction anymore in front of string instruction.
Due to this, OSes that does not clear direction flag in task switch can enter
exception handler with D flag set. This is ABI violation since ABI requires D
flag to b
--- Comment #3 from burnus at gcc dot gnu dot org 2008-04-29 12:03 ---
> const for non function decls, mean nothing to the middle-end.
> I mean on local variables, const mean nothing.
Good to know. Then every (non-VALUE) dummy argument except POINTERs and
ALLOCATABLEs can be marked as "
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-04-29 11:41 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-04-29 11:40 ---
Confirmed. Testcase:
extern void abort (void);
unsigned test (unsigned x)
{
return x / 0x8001U / 0x0002U;
}
int main()
{
if (test(2) != 0)
abort ();
return 0;
}
--
rguenth at gcc dot gnu dot
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-04-29 11:35 ---
I mean on local variables, const mean nothing.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36076
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-04-29 11:35 ---
const for non function decls, mean nothing to the middle-end.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36076
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-04-29 11:30 ---
Please reduce this to a specific revision.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |rguenth at gcc dot gnu dot
|dot org
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Component|middle-end |tree-optimization
Keywords||ice
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-04-29 11:13 ---
Created an attachment (id=15545)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15545&action=view)
proposed patch
Can you try this patch with dealII?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36078
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |rguenth at gcc dot gnu dot
|dot org
--- Comment #5 from Joey dot ye at intel dot com 2008-04-29 10:41 ---
Can be related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36078, where I do
have a small case.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36074
Start from trunk 134730, still fail by 134775:
$ cat f2.f90
subroutine foo(func,p,eval)
real(kind=kind(1.0d0)), dimension(3,0:4,0:4,0:4) :: p
logical(kind=kind(.true.)), dimension(5,5,5) :: eval
interface
subroutine func(values,pt)
real(kind=kind(1.0d0)),
--- Comment #2 from peter at xmos dot com 2008-04-29 10:23 ---
Created an attachment (id=15544)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15544&action=view)
test.i
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36077
--- Comment #1 from peter at xmos dot com 2008-04-29 10:23 ---
Created an attachment (id=15543)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15543&action=view)
test.c
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36077
I am developing a test generator for in house compilers and have come across a
difference where gcc does not agree with our compilers or the Microsoft
compiler.
The result t1_2a of the expression is what differs between gcc and other
compilers.
This is on gcc (GCC) 3.4.6 20060404 (Red Hat 3.4.6-9
--- Comment #6 from rguenth at gcc dot gnu dot org 2008-04-29 10:05 ---
tree-ssa-reassoc can/should be extended to do this.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-04-29 10:01 ---
The ifcombine pass would do this since 4.3.0 if there were not an extra
conversion
for a bittest of bit zero:
:
a.0_2 = (int) a_1(D);
D.1556_3 = a.0_2 & 1;
if (D.1556_3 != 0)
goto (heaven);
else
got
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-04-29 09:55 ---
For the testcase in comment #4 after FRE we have
c_2 = a_1(D) + 4;
d_3 = c_2 * 2;
e_4 = a_1(D) * 2;
f_5 = e_4 + 4;
D.1557_6 = f_5 + d_3;
return D.1557_6;
And a patched reassoc pass then produces
c_2
--- Comment #1 from jakub at gcc dot gnu dot org 2008-04-29 09:19 ---
Testing a fix.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unass
--- Comment #5 from jakub at gcc dot gnu dot org 2008-04-29 09:06 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #3 from jakub at gcc dot gnu dot org 2008-04-29 09:05 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #4 from jakub at gcc dot gnu dot org 2008-04-29 08:59 ---
Subject: Bug 35650
Author: jakub
Date: Tue Apr 29 08:58:58 2008
New Revision: 134789
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134789
Log:
PR c++/35650
* parser.c (cp_parser_lookup_name):
--- Comment #3 from jakub at gcc dot gnu dot org 2008-04-29 08:59 ---
Subject: Bug 35650
Author: jakub
Date: Tue Apr 29 08:58:20 2008
New Revision: 134788
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134788
Log:
PR c++/35650
* parser.c (cp_parser_lookup_name):
--- Comment #2 from jakub at gcc dot gnu dot org 2008-04-29 08:57 ---
Subject: Bug 35987
Author: jakub
Date: Tue Apr 29 08:56:27 2008
New Revision: 134787
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134787
Log:
PR c++/35987
* typeck.c (cp_build_modify_expr) :
--- Comment #1 from jakub at gcc dot gnu dot org 2008-04-29 08:55 ---
Subject: Bug 35987
Author: jakub
Date: Tue Apr 29 08:54:45 2008
New Revision: 134786
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134786
Log:
PR c++/35987
* typeck.c (cp_build_modify_expr) :
--- Comment #7 from aoliva at gcc dot gnu dot org 2008-04-29 08:19 ---
Subject: Bug 35542
Author: aoliva
Date: Tue Apr 29 08:18:36 2008
New Revision: 134782
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=134782
Log:
* fwprop.c (forward_propagate_and_simplify): Replace entire
var
79 matches
Mail list logo
