--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-15 09:49 ---
Seems ix86_function_arg_advance ignores padding for 8 byte aligned arguments
when updating cum-words.
With:
#include stdarg.h
long double
test (int a, int b, int c, int d, int e, int f, int g, int g2, int g3, long
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-15 12:03 ---
Created an attachment (id=21209)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21209action=view)
gcc46-pr44942.patch
Untested fix. Grep tells me that cum-words is only ever used for x86-64
va_start expansion
Version: 4.6.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
GCC target triplet: x86_64-linux
http://gcc.gnu.org/bugzilla
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-15 13:15 ---
If we want to be ABI compatible, I'm afraid it needs to be 16 byte aligned
only.
We don't align aligned(64) structs to 64 bytes either, even with -mavx.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44948
--- Comment #9 from jakub at gcc dot gnu dot org 2010-07-15 15:09 ---
Should be fixed now for 4.6+.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from jakub at gcc dot gnu dot org 2010-07-15 16:28 ---
That is going to break the ABI a lot. You'd e.g. change long double passing
for -m64.
What you IMHO want is to do what the code currently does, and if the alignment
after that is 32 bytes, use a predicate similar
--- Comment #8 from jakub at gcc dot gnu dot org 2010-07-15 20:20 ---
Ah, so -fdump-tree-optimized dump doesn't reveal the problem, but
-fdump-tree-optimized-all actually does. The problem is that each routine -
char_array_structure_constructor and alloc, use different decls
--- Comment #1 from jakub at gcc dot gnu dot org 2010-07-13 09:22 ---
User bug. If you link -lpthread statically (which you really shouldn't, static
linking is highly not recommended), you need to ensure it is linked in as whole
(i.e. -Wl,--whole-archive -lpthread -Wl,--no-whole
--- Comment #8 from jakub at gcc dot gnu dot org 2010-07-13 07:00 ---
Subject: Bug 44901
Author: jakub
Date: Tue Jul 13 06:59:56 2010
New Revision: 162126
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162126
Log:
PR debug/44901
* vec.h (VEC_block_remove): Fix
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-13 12:22 ---
It is not a gcc thing though, but glibc. Some Linux distributions (e.g. recent
Fedora or RHEL) ld -r all libpthread.a objects together, so this works, others
do not.
--
jakub at gcc dot gnu dot org changed
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-13 14:04 ---
Subject: Bug 44701
Author: jakub
Date: Tue Jul 13 14:03:49 2010
New Revision: 162142
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162142
Log:
PR testsuite/44701
* recog.c (constrain_operands
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-13 17:08 ---
Please don't keep reopening this bug.
The symbols are weak undefs because libgfortran doesn't require (and shouldn't
require) libpthread, it is thread-safe only when libpthread is linked in.
--
jakub at gcc dot
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-12 10:19 ---
Fails even with -fno-var-tracking, so does not look VTA related.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44901
--- Comment #5 from jakub at gcc dot gnu dot org 2010-07-12 10:38 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-12 14:26 ---
Created an attachment (id=21183)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21183action=view)
pr44901.i
Reduced testcase:
./xgcc -B ./ -S -m32 -fcompare-debug -O2 -fno-var-tracking /tmp/pr44901.i -w
xgcc
--- Comment #5 from jakub at gcc dot gnu dot org 2010-07-12 14:50 ---
All the unordered removals for local_decls are obviously wrong.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44901
--- Comment #7 from jakub at gcc dot gnu dot org 2010-07-12 15:23 ---
Created an attachment (id=21185)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21185action=view)
gcc46-pr44901.patch
Untested fix.
Expansion depends on the order of vars in cfun-local_decls, see e.g
--- Comment #1 from jakub at gcc dot gnu dot org 2010-07-11 15:32 ---
That means the configuration of your hobby target doesn't define
NO_IMPLICIT_EXTERN_C and thus all system headers are considered to be
implicitly surrounded by extern C.
--
jakub at gcc dot gnu dot org changed
--- Comment #1 from jakub at gcc dot gnu dot org 2010-07-10 11:26 ---
gcc 4.5+ requires gdb 7.0+, so I'd say this is a user error.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44898
--- Comment #20 from jakub at gcc dot gnu dot org 2010-07-09 17:12 ---
Created an attachment (id=21162)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21162action=view)
gcc46-pr28632.patch
Sorry, I haven't been aware of this PR. The attached patch brings in some of
the ideas from
--- Comment #21 from jakub at gcc dot gnu dot org 2010-07-09 19:40 ---
Subject: Bug 28632
Author: jakub
Date: Fri Jul 9 19:40:03 2010
New Revision: 162009
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162009
Log:
PR tree-optimization/28632
* tree-vrp.c
--- Comment #10 from jakub at gcc dot gnu dot org 2010-07-08 07:47 ---
I'd say the test instead just should use signed char instead of char.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44828
--- Comment #7 from jakub at gcc dot gnu dot org 2010-07-08 07:54 ---
Currently libgomp terminates all threads immediately, unless mandated by the
standard (i.e. in non-nested case the threads need to be kept around until next
omp parallel).
Yes, it would be possible to keep other
--
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 #1 from jakub at gcc dot gnu dot org 2010-07-08 08:37 ---
Things go wrong in the combiner:
(insn 21 42 22 6 pr44858.c:16 (parallel [
(set (reg/v:SI 62 [ c ])
(and:SI (reg:SI 68)
(const_int 2 [0x2])))
(clobber
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-08 09:58 ---
Caused by http://gcc.gnu.org/viewcvs?root=gccview=revrev=152642
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-08 10:01 ---
Updated testcase:
extern void abort (void);
int a = 3;
int b = 1;
__attribute__((noinline)) long long
foo (int x, int y)
{
return x / y;
}
__attribute__((noinline)) int
bar (void)
{
int c = 2;
c = foo (1, b
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-08 11:14 ---
Created an attachment (id=21140)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21140action=view)
gcc46-pr44847.patch
Untested fix. Thanks for the report.
--
jakub at gcc dot gnu dot org changed
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-08 14:44 ---
Created an attachment (id=21147)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21147action=view)
gcc46-vrp.patch
So, combiner seems to be the first to notice that c must be actually zero
(2 (x y)).
This patch
--- Comment #11 from jakub at gcc dot gnu dot org 2010-07-08 17:08 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-08 17:31 ---
Subject: Bug 44847
Author: jakub
Date: Thu Jul 8 17:30:41 2010
New Revision: 161967
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161967
Log:
PR fortran/44847
* match.c (match_exit_cycle
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-08 17:33 ---
Subject: Bug 44847
Author: jakub
Date: Thu Jul 8 17:33:01 2010
New Revision: 161968
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161968
Log:
PR fortran/44847
* match.c (match_exit_cycle
--- Comment #5 from jakub at gcc dot gnu dot org 2010-07-08 17:35 ---
Subject: Bug 44847
Author: jakub
Date: Thu Jul 8 17:35:32 2010
New Revision: 161969
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161969
Log:
PR fortran/44847
* match.c (match_exit_cycle
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-08 18:13 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-08 20:01 ---
Then you should probably just
gcc_assert (POINTER_TYPE_P (TREE_TYPE (arg));
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44877
--- Comment #5 from jakub at gcc dot gnu dot org 2010-07-07 07:17 ---
Yeah, the amount of regressions is huge, both on x86_64-linux and i686-linux.
The difference is in i386 now overriding config/elfos.h definition:
/* Write the extra assembler code needed to declare a function
--- Comment #15 from jakub at gcc dot gnu dot org 2010-07-07 13:23 ---
The reduced testcase doesn't fail for me, the unreduced one fails though.
-fcompare-debug complains about different ORIGINAL_REGNOs and different ivtmp.*
names.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--- Comment #18 from jakub at gcc dot gnu dot org 2010-07-07 13:55 ---
That's not SSA name, but DECL_UID of the underlying decl.
I believe such changes are considered ok if it just means different gaps
between uids with -g vs. -g0.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--- Comment #23 from jakub at gcc dot gnu dot org 2010-07-07 14:25 ---
Hash tables hashed by DECL_UID should be traversed only when the traversal
order doesn't affect code generation. E.g. Richard fixed a few places to just
set
bits in a bitmap and then iterate over the bitmap for code
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-06 08:21 ---
That's how GOMP_CPU_AFFINITY works - it is a round-robin assignment of CPUs
upon thread creation, and the first two threads are never recreated in this
case.
Currently there is no tracking on how many threads bind
--- Comment #1 from jakub at gcc dot gnu dot org 2010-07-06 09:25 ---
Shorter testcase:
extern void abort (void);
static char
foo (char si1, char si2)
{
return si1 * si2;
}
int a = 0x105F61CA;
int
main (void)
{
int b = 0x0332F5C8;
if (foo (b, a) 0)
abort ();
return 0
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-06 09:33 ---
Not sure whether the testcase is valid or not. The multiplication using char
variables on both sides (and likewise for result) is: -54 * -56 (= 3024),
but (char) 3024 is -48. For int that would be clear undefined
--- Comment #1 from jakub at gcc dot gnu dot org 2010-07-06 10:42 ---
Guess sparc needs similar fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44834
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-05 10:55 ---
I don't see anything odd on the value-expr (the variable res has been NRVed in
the FE to the RESULT_DECL) nor on the assignments from RESULT_DECL.
So, I think either we should guard this hunk in gimplify.c
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-05 11:04 ---
Created an attachment (id=21091)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21091action=view)
gcc46-pr44808.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44808
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-05 12:18 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #11 from jakub at gcc dot gnu dot org 2010-07-05 12:21 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #4 from jakub at gcc dot gnu dot org 2010-07-05 14:42 ---
Subject: Bug 44808
Author: jakub
Date: Mon Jul 5 14:42:20 2010
New Revision: 161838
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161838
Log:
PR c++/44808
* gimplify.c (gimplify_modify_expr
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-05 18:00 ---
You need 32-bit HWI host to reproduce.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44825
--- Comment #2 from jakub at gcc dot gnu dot org 2010-07-02 09:15 ---
Created an attachment (id=21063)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21063action=view)
gcc46-pr44780.patch
Untested fix.
--
jakub at gcc dot gnu dot org changed:
What|Removed
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-02 09:22 ---
Then the right fix would be not to assume that all such options start with -W,
no?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44774
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-02 13:22 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #3 from jakub at gcc dot gnu dot org 2010-07-02 20:22 ---
Subject: Bug 44780
Author: jakub
Date: Fri Jul 2 20:22:32 2010
New Revision: 161742
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161742
Log:
PR c++/44780
* typeck.c (convert_for_assignment
--- Comment #15 from jakub at gcc dot gnu dot org 2010-07-01 07:13 ---
Created an attachment (id=21052)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21052action=view)
gcc46-pr44694-arm.patch
Untested fix for this issue.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44694
--- Comment #16 from jakub at gcc dot gnu dot org 2010-07-01 07:24 ---
Created an attachment (id=21053)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21053action=view)
gcc46-pr44694-arm.patch
Actually that was wrong, we need to emit DW_OP_fbreg offset DW_OP_stack_value
--- Comment #5 from jakub at gcc dot gnu dot org 2010-07-01 11:02 ---
Subject: Bug 44575
Author: jakub
Date: Thu Jul 1 11:01:58 2010
New Revision: 161660
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161660
Log:
Backport from mainline
2010-06-21 Jakub Jelinek
--- Comment #8 from jakub at gcc dot gnu dot org 2010-07-01 11:03 ---
Subject: Bug 43801
Author: jakub
Date: Thu Jul 1 11:03:31 2010
New Revision: 161661
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161661
Log:
Backport from mainline
2010-06-29 Jakub Jelinek
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-01 11:06 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #9 from jakub at gcc dot gnu dot org 2010-07-01 11:07 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #17 from jakub at gcc dot gnu dot org 2010-07-01 11:12 ---
Subject: Bug 44694
Author: jakub
Date: Thu Jul 1 11:11:46 2010
New Revision: 161662
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161662
Log:
PR debug/44694
* dwarf2out.c (reg_loc_descriptor
--- Comment #18 from jakub at gcc dot gnu dot org 2010-07-01 11:13 ---
Subject: Bug 44694
Author: jakub
Date: Thu Jul 1 11:12:24 2010
New Revision: 161663
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161663
Log:
PR debug/44694
* dwarf2out.c (reg_loc_descriptor
--- Comment #9 from jakub at gcc dot gnu dot org 2010-07-01 12:21 ---
The important question is if the testcase still ICEs with the fix reverted when
you do that change.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44592
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-01 12:31 ---
Subject: Bug 44104
Author: jakub
Date: Thu Jul 1 12:31:19 2010
New Revision: 161666
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161666
Log:
2010-07-01 Jakub Jelinek ja...@redhat.com
Backport from
--- Comment #19 from jakub at gcc dot gnu dot org 2010-07-01 12:37 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #6 from jakub at gcc dot gnu dot org 2010-07-01 12:47 ---
Subject: Bug 40421
Author: jakub
Date: Thu Jul 1 12:47:26 2010
New Revision: 161668
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161668
Log:
Backport from mainline
2009-06-13 Richard
--- Comment #7 from jakub at gcc dot gnu dot org 2010-07-01 12:54 ---
Subject: Bug 40421
Author: jakub
Date: Thu Jul 1 12:53:44 2010
New Revision: 161669
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161669
Log:
PR tree-optimization/40421
* gfortran.fortran
--- Comment #8 from jakub at gcc dot gnu dot org 2010-07-01 12:55 ---
Subject: Bug 40421
Author: jakub
Date: Thu Jul 1 12:55:28 2010
New Revision: 161670
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161670
Log:
PR tree-optimization/40421
* gfortran.fortran
--- Comment #9 from jakub at gcc dot gnu dot org 2010-06-30 06:12 ---
Subject: Bug 44694
Author: jakub
Date: Wed Jun 30 06:12:22 2010
New Revision: 161587
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161587
Log:
PR debug/44694
* cselib.h
--- Comment #10 from jakub at gcc dot gnu dot org 2010-06-30 07:16 ---
Fixed on the trunk so far.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #10 from jakub at gcc dot gnu dot org 2010-06-30 15:17 ---
Subject: Bug 44059
Author: jakub
Date: Wed Jun 30 15:16:54 2010
New Revision: 161607
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161607
Log:
Backport from mainline
2010-05-11 Jakub Jelinek
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-30 15:23 ---
Subject: Bug 42278
Author: jakub
Date: Wed Jun 30 15:23:13 2010
New Revision: 161608
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161608
Log:
Backport from mainline
2010-05-12 Jakub Jelinek
--- Comment #24 from jakub at gcc dot gnu dot org 2010-06-30 15:34 ---
Subject: Bug 43866
Author: jakub
Date: Wed Jun 30 15:34:09 2010
New Revision: 161609
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161609
Log:
Backport from mainline
2010-06-25 Jakub Jelinek
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-30 15:41 ---
Subject: Bug 44694
Author: jakub
Date: Wed Jun 30 15:40:53 2010
New Revision: 161610
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161610
Log:
Backport from mainline
2010-06-30 Jakub Jelinek
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-29 06:28 ---
The extremely long location lists are caused mainly by reverse_op created
equivalences. Wonder whether we shouldn't on RHS replace sp and sp + const_int
with framep and framep + const_int even outside of memory
ReportedBy: jakub at gcc dot gnu dot org
GCC target triplet: powerpc-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44707
--- Comment #6 from jakub at gcc dot gnu dot org 2010-06-29 08:53 ---
Created an attachment (id=21035)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21035action=view)
gcc46-pr44694.patch
With this patchlet the compile time on ginac.ii went down from more than 3
minutes to 16
--- Comment #7 from jakub at gcc dot gnu dot org 2010-06-29 09:09 ---
readelf -wo ginac.o1 | grep 'End of' | wc -l; readelf -wo ginac.o2 | grep 'End
of' | wc -l; readelf -wo ginac.o1 | wc -l; readelf -wo ginac.o2 | wc -l;
readelf -wo ginac.o1 | grep fbreg | wc -l; readelf -wo ginac.o2
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-29 09:49 ---
Guess something like:
--- gcc/testsuite/gcc.target/powerpc/asm-es-2.c 2009-07-20 20:41:46.0
+0200
+++ gcc/testsuite/gcc.target/powerpc/asm-es-2.c 2010-06-29 11:37:05.482959217
+0200
@@ -11,7 +11,7 @@ f2 (int
Severity: normal
Priority: P3
Component: debug
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44712
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-29 14:11 ---
Created an attachment (id=21039)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21039action=view)
gcc46-pr44694.patch
Updated patch that actually passed bootstrap/regtested on x86_64-linux and
i686-linux. When
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-29 15:24 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #7 from jakub at gcc dot gnu dot org 2010-06-29 15:43 ---
Fixed then.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #6 from jakub at gcc dot gnu dot org 2010-06-29 16:27 ---
Created an attachment (id=21040)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21040action=view)
gcc46-pr43801.patch
Untested fix.
I think for !DECL_ONE_ONLY decls we can put the virtual clone in the same
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-29 16:28 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-29 16:30 ---
Fixed on the trunk, unfixed on the branch.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-29 19:37 ---
Subject: Bug 44668
Author: jakub
Date: Tue Jun 29 19:37:46 2010
New Revision: 161563
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161563
Log:
PR debug/44668
* dwarf2out.c
--- Comment #7 from jakub at gcc dot gnu dot org 2010-06-29 19:46 ---
Subject: Bug 43801
Author: jakub
Date: Tue Jun 29 19:46:05 2010
New Revision: 161564
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161564
Log:
PR tree-optimization/43801
* cgraph.c
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-28 08:54 ---
Created an attachment (id=21021)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21021action=view)
gcc46-pr44682.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44682
--- Comment #24 from jakub at gcc dot gnu dot org 2010-06-28 11:05 ---
On:
extern C void abort ();
struct S
{
int x;
int y;
};
struct T
{
int x;
int y;
T (int u, int v) : x (u), y (v) {}
};
extern const S s;
extern const T t, u;
int sx = s.x;
int tx = t.x;
const S s = { 1
--- Comment #25 from jakub at gcc dot gnu dot org 2010-06-28 11:22 ---
I guess best would be to wait for the constexpr work, then use that as an
infrastructure to discover ctors that aren't marked as constexpr, but they
could be and use that at bit together with !TREE_USED during
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-28 16:12 ---
Seems the problem here is extremely long loc_chain lists (the longest one in
insert_into_intersection has 3740 entries), so e.g. insert_into_intersection is
extremely time consuming. Most of the locations in the long
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-28 20:13 ---
Subject: Bug 44682
Author: jakub
Date: Mon Jun 28 20:12:31 2010
New Revision: 161511
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161511
Log:
PR c++/44682
* class.c (build_base_path
--- Comment #15 from jakub at gcc dot gnu dot org 2010-06-25 08:13 ---
Created an attachment (id=21000)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21000action=view)
gcc46-pr43866.patch
Here is a complete fix.
This includes what the earlier two patches did, plus for nested
--- Comment #18 from jakub at gcc dot gnu dot org 2010-06-25 09:14 ---
That part comes from the questionable testcase, which does
a_sp = matrix%local_data_sp
before the loop unconditionally, eventhough matrix%local_data_sp is
uninitialized unless use_sp is .true.
Without the unswitching
--- Comment #19 from jakub at gcc dot gnu dot org 2010-06-25 09:19 ---
The compiler doesn't know that matrix%local_data_sp is uninitialized (at least
unless it would propagate that info in some IPA pass), matrix%local_data_sp is
a field in what an arguments points at (and the argument
--- Comment #21 from jakub at gcc dot gnu dot org 2010-06-25 09:38 ---
True. Most of the fields of the pointer are still uninitialized (NULLIFY only
clears the data field).
I'm afraid when NULLIFY doesn't clear the whole struct there will be always a
possibility valgrind might complain
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-25 11:26 ---
This goes wrong with -O2 -funroll-loops already during tree opts. In vrp2 we
have:
b[2][1]{lb: 1 sz: 1} = D.1599_148;
b[3][1]{lb: 1 sz: 1} = D.1599_154;
D.1635_152 = 2;
D.1636_151 = b[2][1]{lb: 1 sz: 1};
D
--- Comment #6 from jakub at gcc dot gnu dot org 2010-06-25 12:07 ---
Subject: Bug 44539
Author: jakub
Date: Fri Jun 25 12:07:00 2010
New Revision: 161374
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161374
Log:
PR tree-optimization/44539
* tree-cfgcleanup.c
--- Comment #22 from jakub at gcc dot gnu dot org 2010-06-25 12:11 ---
Subject: Bug 43866
Author: jakub
Date: Fri Jun 25 12:10:42 2010
New Revision: 161375
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161375
Log:
PR middle-end/43866
* tree-ssa-loop-unswitch.c
--- Comment #7 from jakub at gcc dot gnu dot org 2010-06-25 12:30 ---
Should be fixed now.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
401 - 500 of 6780 matches
Mail list logo