--
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-09-08 14:02 ---
Created an attachment (id=21738)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21738action=view)
gcc46-pr45595.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45595
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-08 14:25 ---
Created an attachment (id=21739)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21739action=view)
gcc46-pr45597.patch
Seems to be a recent regression, caused by
http://gcc.gnu.org/viewcvs?view=revisionrevision
--- Comment #13 from jakub at gcc dot gnu dot org 2010-09-08 14:48 ---
And, getdelim insist on allocating resp. reallocating the buffer. That is of
course usually desirable when used from C, but I'm afraid it isn't in this
case, where you have user provided buffer with fixed size. You
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-08 16:19 ---
Yes, please, and assign to me (working on a simplify_comparison fix for that).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20517
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-08 16:46 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 16:46:13 2010
New Revision: 164004
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164004
Log:
PR fortran/45595
* openmp.c (resolve_omp_do
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-08 16:47 ---
Subject: Bug 45597
Author: jakub
Date: Wed Sep 8 16:47:16 2010
New Revision: 164005
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164005
Log:
PR fortran/45597
* trans-openmp.c
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-08 17:22 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 17:22:36 2010
New Revision: 164007
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164007
Log:
PR fortran/45595
* openmp.c (resolve_omp_do
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-08 17:24 ---
Subject: Bug 45595
Author: jakub
Date: Wed Sep 8 17:23:52 2010
New Revision: 164008
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164008
Log:
PR fortran/45595
* openmp.c (resolve_omp_do
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-08 17:52 ---
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-09-08 17:52 ---
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-09-08 18:10 ---
Created an attachment (id=21744)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21744action=view)
gcc46-pr20517.patch
Untested patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20517
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-07 08:05 ---
This isn't specific to Fortran,
struct S { int a; };
struct S foo (struct S *arg)
{
struct S s;
if (bar ())
baz (not valid);
else
s = *arg;
return s;
}
has the exactly same difference.
I believe
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-07 22:45 ---
Reproduced with:
void bar (unsigned char *);
template int N
struct S
{
static const int k = 6;
};
template int N
const int SN::k;
template int N
void foo ()
{
const int i = SN::k;
unsigned char a[i];
bar
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-07 22:46 ---
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-09-06 06:33 ---
In G++ 4.6+ you can do:
int i;
unsigned j;
template class T
void
bar (const T t)
{
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored -Wtype-limits
if (t 0)
#pragma GCC diagnostic pop
{
i
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-06 09:48 ---
Subject: Bug 45543
Author: jakub
Date: Mon Sep 6 09:48:10 2010
New Revision: 163907
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163907
Log:
PR testsuite/45543
* g++.dg/debug/dwarf2/typedef1
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-04 08:01 ---
*** This bug has been marked as a duplicate of 45519 ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from jakub at gcc dot gnu dot org 2010-09-04 08:01 ---
*** Bug 45533 has been marked as a duplicate of this bug. ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-03 07:30 ---
I think for -D and -U at least the canonicalization should be joined instead of
separate, because that's how people are used to write these options (which
doesn't mean the tools the driver invokes shouldn't be changed
--- Comment #8 from jakub at gcc dot gnu dot org 2010-09-03 09:10 ---
Seems http://gcc.gnu.org/viewcvs?root=gccview=revrev=162842
has been committed, just Honza forgot to mention the PR in the ChangeLog,
testcase and commit message.
I certainly can't reproduce this bug with 20100811 gcc
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-03 09:54 ---
Subject: Bug 45500
Author: jakub
Date: Fri Sep 3 09:53:44 2010
New Revision: 163803
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163803
Log:
PR debug/45500
* dwarf2out.c (rtl_for_decl_init
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-03 10:15 ---
Subject: Bug 45500
Author: jakub
Date: Fri Sep 3 10:14:43 2010
New Revision: 163807
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163807
Log:
PR debug/45500
* dwarf2out.c (rtl_for_decl_init
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-03 10:50 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
ReportedBy: jakub at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45517
--- Comment #23 from jakub at gcc dot gnu dot org 2010-09-03 14:47 ---
Subject: Bug 45484
Author: jakub
Date: Fri Sep 3 14:46:39 2010
New Revision: 163823
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163823
Log:
PR middle-end/45484
* dwarf2out.c
--- Comment #24 from jakub at gcc dot gnu dot org 2010-09-03 14:50 ---
Subject: Bug 45484
Author: jakub
Date: Fri Sep 3 14:50:20 2010
New Revision: 163824
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163824
Log:
PR middle-end/45484
* dwarf2out.c
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-03 15:44 ---
*** Bug 45517 has been marked as a duplicate of this bug. ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-03 15:44 ---
*** This bug has been marked as a duplicate of 41063 ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from jakub at gcc dot gnu dot org 2010-09-03 20:53 ---
I don't see anything confusing about it. If you have:
union { int a, b, c; } u;
u.a overlaps u.b and u.c as well, and the same applies to bitfields. union
isn't struct, see ISO C99 6.7.2.1/5. The standard is clear
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-02 07:57 ---
Subject: Bug 45458
Author: jakub
Date: Thu Sep 2 07:56:48 2010
New Revision: 163761
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163761
Log:
PR middle-end/45458
* bb-reorder.c
--- Comment #4 from jakub at gcc dot gnu dot org 2010-09-02 08:01 ---
Subject: Bug 45423
Author: jakub
Date: Thu Sep 2 08:00:55 2010
New Revision: 163762
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163762
Log:
Backport from mainline
2010-08-30 Jakub Jelinek
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-02 08:01 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-02 08:02 ---
Fixed for GCC 4.4+.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-02 10:48 ---
Why are you using -fvar-tracking-assignments together with selective
scheduling?
It is known not to work (and that's why it isn't enabled with -g).
--
jakub at gcc dot gnu dot org changed:
What
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-02 11:51 ---
This isn't a testsuite bug, but lto/whopr bug, and IMHO quite severe, it
affects debugging even at -O0.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44989
--- Comment #6 from jakub at gcc dot gnu dot org 2010-09-02 12:37 ---
Seems to be a bug in free_lang_data_in_block, which throws away the
DECL_EXTERNAL decl from BLOCK_VARS, eventhough it is obviously very much
needed. In fact, is there ever a case where what free_lang_data_in_block
--- Comment #21 from jakub at gcc dot gnu dot org 2010-09-02 16:16 ---
Created an attachment (id=21670)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21670action=view)
gcc46-pr45484.patch
The last valgrind error is my fault, we don't need a BARRIER there, any
instruction
--
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 #2 from jakub at gcc dot gnu dot org 2010-09-02 18:33 ---
Created an attachment (id=21677)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21677action=view)
gcc46-pr45500.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45500
--- Comment #1 from jakub at gcc dot gnu dot org 2010-09-02 21:47 ---
I guess this is related to the recent option handling changes. At least, it
worked fine yesterday and doesn't work today (I only see a-strunb.ads issues
myself).
--
jakub at gcc dot gnu dot org changed
--- Comment #33 from jakub at gcc dot gnu dot org 2010-09-01 11:50 ---
Yes, 4.4.5 and maybe 4.4.6 is planned.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44328
--
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-09-01 16:38 ---
*** This bug has been marked as a duplicate of 43706 ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #21 from jakub at gcc dot gnu dot org 2010-09-01 16:38 ---
*** Bug 45485 has been marked as a duplicate of this bug. ***
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from jakub at gcc dot gnu dot org 2010-09-01 17:12 ---
Created an attachment (id=21648)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21648action=view)
gcc46-pr45458.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45458
--- Comment #13 from jakub at gcc dot gnu dot org 2010-09-01 18:32 ---
Yeah, if the bug is still present on the trunk, it definitely should be fixed
on the trunk first, and whether it is actually backportable or not is to be
decided afterwards. We just happen to have a reproducer only
--- Comment #3 from jakub at gcc dot gnu dot org 2010-09-01 19:46 ---
Subject: Bug 45458
Author: jakub
Date: Wed Sep 1 19:46:21 2010
New Revision: 163743
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163743
Log:
PR middle-end/45458
* bb-reorder.c
--
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-08-31 11:24 ---
Created an attachment (id=21601)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21601action=view)
gcc46-pr45457.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45457
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-31 11:55 ---
Created an attachment (id=21604)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21604action=view)
gcc46-pr45461.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45461
--- Comment #4 from jakub at gcc dot gnu dot org 2010-08-31 16:13 ---
Subject: Bug 45461
Author: jakub
Date: Tue Aug 31 16:13:14 2010
New Revision: 163678
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163678
Log:
PR middle-end/45461
* builtins.c (dummy_object
--- Comment #5 from jakub at gcc dot gnu dot org 2010-08-31 16:47 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-31 19:48 ---
Created an attachment (id=21614)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21614action=view)
gcc46-pr45250.patch
The problem is that the PA backend has quite lame setup, where
FRAME_POINTER_REGNUM
--- Comment #5 from jakub at gcc dot gnu dot org 2010-08-31 20:13 ---
That's because the whole foo function doesn't have any side-effects, so it is
optimized away completely.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45467
--- Comment #11 from jakub at gcc dot gnu dot org 2010-08-31 20:53 ---
Because when foo is not static, it has to be compiled. If it is static, GCC
figures it is a pure function (only reads memory and arguments and computes
from it its return value) and as the result in main
--- Comment #17 from jakub at gcc dot gnu dot org 2010-08-31 22:26 ---
Manuel, can you back up your claims about the C FE being slow with some
numbers? I don't remember the C FE ever being a time issue recently, of course
C++ is a different story.
--
http://gcc.gnu.org/bugzilla
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-31 22:47 ---
Subject: Bug 45457
Author: jakub
Date: Tue Aug 31 22:47:25 2010
New Revision: 163705
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163705
Log:
PR preprocessor/45457
* expr.c (parse_defined
--- Comment #3 from jakub at gcc dot gnu dot org 2010-08-31 23:23 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #1 from jakub at gcc dot gnu dot org 2010-08-30 11:38 ---
Created an attachment (id=21591)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21591action=view)
gcc46-pr45423.patch
Patch I'm going to bootstrap/regtest.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-30 12:53 ---
Created an attachment (id=21593)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21593action=view)
gcc46-pr45419.patch
Fix for the valgrind issue.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45419
--- Comment #8 from jakub at gcc dot gnu dot org 2010-08-30 14:05 ---
The other issue comes from MEM_ATTRS caching.
mem_attrs_htab_eq
considers MEM_EXPR to be equal if:
279 (p-expr == q-expr
280 || (p-expr != NULL_TREE q-expr != NULL_TREE
281 operand_equal_p (p-expr, q-expr, 0
--- Comment #9 from jakub at gcc dot gnu dot org 2010-08-30 14:29 ---
Unfortunately printing iterative_hash_expr for MEM_EXPR if final_insns_dump_p
is actually worse, because iterative_hash_expr uses DECL_UID, which isn't
guaranteed to be the same between -g and -g0.
So, either we
--- Comment #4 from jakub at gcc dot gnu dot org 2010-08-30 16:06 ---
Please provide preprocessed source. Sounds like my patch uncovered a latent
issue.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-30 17:16 ---
Subject: Bug 45423
Author: jakub
Date: Mon Aug 30 17:16:29 2010
New Revision: 163653
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163653
Log:
PR middle-end/45423
* gimplify.c
--- Comment #11 from jakub at gcc dot gnu dot org 2010-08-30 17:17 ---
Subject: Bug 45419
Author: jakub
Date: Mon Aug 30 17:17:15 2010
New Revision: 163654
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163654
Log:
PR debug/45419
* tree-ssa-live.c
--- Comment #6 from jakub at gcc dot gnu dot org 2010-08-30 19:32 ---
Subject: Bug 45079
Author: jakub
Date: Mon Aug 30 19:31:45 2010
New Revision: 163657
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163657
Log:
Backport from mainline
2010-07-27 Jakub Jelinek
--- Comment #3 from jakub at gcc dot gnu dot org 2010-08-30 19:33 ---
Subject: Bug 45423
Author: jakub
Date: Mon Aug 30 19:33:31 2010
New Revision: 163658
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163658
Log:
PR middle-end/45423
* gimplify.c
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-30 19:33 ---
Fixed, sorry for the delay.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--
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 #3 from jakub at gcc dot gnu dot org 2010-08-28 10:02 ---
This is invalid.
See OpenMP 3.0, 2.9.4.1, COPYIN restrictions on page 102, lines 17-18:
An array with the ALLOCATABLE attribute must be in the allocated state. Each
thread's copy of that array must be allocated
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-27 21:35 ---
You need to read the documentation, which would tell you you need to use
-fmain=HelloWorld option.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from jakub at gcc dot gnu dot org 2010-08-26 07:14 ---
It isn't in terrible shape on many targets, it is only on targets with very
small amount of registers where -fschedule-insns didn't in 4.4 and earlier
(since forever) work reliably and wasn't enabled by default either
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-26 07:33 ---
That's not the main problem here. The problem is that this is an aggregate and
we SRA only automatic vars, not global vars. And it isn't constant either
(neither declared, nor actually, so can't be detected
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-26 09:23 ---
There of course are
-Wno-unused-{function,label,parameter,value,variable,but-set-{parameter,variable}}
options.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45411
--- Comment #4 from jakub at gcc dot gnu dot org 2010-08-26 09:59 ---
Sure, one that would also be able to assume that main is called just once.
Anyway, even to optimize ss.i into 77 you'd need a way to mark fields in global
aggregates as const if they aren't ever modified
--- Comment #8 from jakub at gcc dot gnu dot org 2010-08-26 16:39 ---
Subject: Bug 44485
Author: jakub
Date: Thu Aug 26 16:39:26 2010
New Revision: 163568
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163568
Log:
PR tree-optimization/44485
* calls.c
--- Comment #9 from jakub at gcc dot gnu dot org 2010-08-26 16:48 ---
Should be fixed now.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from jakub at gcc dot gnu dot org 2010-08-26 17:58 ---
Unfortunately can't reproduce it myself, neither with bootstrapped, nor with
just stage1 built gcc.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45419
Version: 4.6.0
Status: UNCONFIRMED
Keywords: openmp
Severity: normal
Priority: P3
Component: middle-end
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
--
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 #6 from jakub at gcc dot gnu dot org 2010-08-25 09:29 ---
Created an attachment (id=21559)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21559action=view)
gcc46-pr45400.patch
Untested fix.
I think it is already wrong to go through SUBREG that is changing mode class
--- Comment #6 from jakub at gcc dot gnu dot org 2010-08-25 09:37 ---
Subject: Bug 45059
Author: jakub
Date: Wed Aug 25 09:36:35 2010
New Revision: 163539
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163539
Log:
PR tree-optimization/45059
* tree-ssa-reassoc.c
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-25 09:50 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #24 from jakub at gcc dot gnu dot org 2010-08-25 09:53 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|WAITING
--- Comment #2 from jakub at gcc dot gnu dot org 2010-08-25 10:00 ---
Caused by r140120 (i.e. PR37302), guess the declaration is now added during
tentative parsing which is then not committed and parsed again differently,
defining the parameter again.
--
jakub at gcc dot gnu dot org
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-25 13:25 ---
Created an attachment (id=21560)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21560action=view)
gcc46-pr44858.patch
The combiner bug seems to be similar to PR20322, attached untested patch should
fix
--- Comment #8 from jakub at gcc dot gnu dot org 2010-08-25 14:01 ---
Created an attachment (id=21561)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21561action=view)
gcc46-pr44858-2.patch
Alternate patch, which doesn't give up so easily (usually both sets can go in
any order, so
--- Comment #6 from jakub at gcc dot gnu dot org 2010-08-25 16:39 ---
The problem is that func_21 is first marked as
TREE_READONLY/DECL_LOOPING_CONST_OR_PURE_P by local_pure_const, thus there are
no vops on it, but afterwards it is by another local_pure_const made
also
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Priority|P3 |P4
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44415
--- Comment #10 from jakub at gcc dot gnu dot org 2010-08-25 16:48 ---
The patch has been approved, why hasn't been applied?
http://gcc.gnu.org/ml/gcc-patches/2010-07/msg00381.html
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-25 17:49 ---
Subject: Bug 45400
Author: jakub
Date: Wed Aug 25 17:49:26 2010
New Revision: 163551
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163551
Log:
PR rtl-optimization/45400
* combine.c
--- Comment #9 from jakub at gcc dot gnu dot org 2010-08-25 17:51 ---
Subject: Bug 44858
Author: jakub
Date: Wed Aug 25 17:50:59 2010
New Revision: 163552
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163552
Log:
PR rtl-optimization/44858
* combine.c (try_combine
--- Comment #8 from jakub at gcc dot gnu dot org 2010-08-25 21:24 ---
Subject: Bug 45400
Author: jakub
Date: Wed Aug 25 21:23:57 2010
New Revision: 163554
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163554
Log:
PR rtl-optimization/45400
* combine.c
--- Comment #10 from jakub at gcc dot gnu dot org 2010-08-25 21:25 ---
Subject: Bug 44858
Author: jakub
Date: Wed Aug 25 21:25:18 2010
New Revision: 163555
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163555
Log:
PR rtl-optimization/44858
* combine.c
--- Comment #9 from jakub at gcc dot gnu dot org 2010-08-25 21:27 ---
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-08-25 21:27 ---
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-08-24 10:18 ---
Created an attachment (id=21554)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21554action=view)
pageattr.i
Compiling this on x86_64-linux (and likely i?86-linux) with 4.4 branch
(r163492) using
./cc1 -fno
--- Comment #10 from jakub at gcc dot gnu dot org 2010-08-24 10:30 ---
It seems things go wrong during RA. In *.asmcons we still have the correct:
...
(insn 672 304 837 3
/mnt/b1/src/linux/set64/arch/x86/include/asm/cmpxchg_32.h:96 (parallel [
(set (reg/f:SI 558 [ value.59
--- Comment #11 from jakub at gcc dot gnu dot org 2010-08-24 10:43 ---
Reloads for insn # 675
Reload 0: reload_in (SI) = (reg/v/f:SI 132 [ kpte ])
GENERAL_REGS, RELOAD_OTHER (opnum = 0)
reload_in_reg: (reg/v/f:SI 132 [ kpte ])
reload_reg_rtx: (reg:SI 5 di)
Reload
101 - 200 of 6780 matches
Mail list logo