--- Comment #5 from dberlin at gcc dot gnu dot org 2009-03-19 17:10 ---
I agree with pinskia, and think this should be closed as invalid.
fargument-noalias-* is an assertion by you, the user, that these things will
never alias.
You are telling the compiler it is allowed to assume
--- Comment #9 from dberlin at gcc dot gnu dot org 2009-02-05 07:10 ---
Subject: Bug 39100
Author: dberlin
Date: Thu Feb 5 07:09:44 2009
New Revision: 143951
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143951
Log:
2009-02-05 Daniel Berlin dber...@dberlin.org
--- Comment #3 from dberlin at gcc dot gnu dot org 2009-02-02 19:42 ---
Eyeballing this, I think y should not end up empty anyway.
Shouldn't it have i in it's points-to set?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39074
--- Comment #12 from dberlin at gcc dot gnu dot org 2008-12-10 20:15
---
Subject: Bug 36792
Author: dberlin
Date: Wed Dec 10 20:13:39 2008
New Revision: 142659
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142659
Log:
2008-12-10 Daniel Berlin [EMAIL PROTECTED]
PR
--- Comment #2 from dberlin at gcc dot gnu dot org 2008-11-21 16:55 ---
(In reply to comment #0)
For this function:
int test (int a, int b, int c, int g)
{
int d, e;
if (a)
d = b * c;
else
d = b - c;
e = b * c + g;
return d + e;
}
the multiply expression
--- Comment #3 from dberlin at gcc dot gnu dot org 2008-07-10 15:01 ---
We need to call loop_optimizer_finalize on the early exit from PRE case.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36788
--- Comment #3 from dberlin at gcc dot gnu dot org 2008-07-08 16:11 ---
Subject: Bug 35287
Author: dberlin
Date: Tue Jul 8 16:11:06 2008
New Revision: 137631
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=137631
Log:
2008-07-05 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #4 from dberlin at gcc dot gnu dot org 2008-07-08 16:11 ---
Subject: Bug 35286
Author: dberlin
Date: Tue Jul 8 16:11:06 2008
New Revision: 137631
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=137631
Log:
2008-07-05 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #16 from dberlin at gcc dot gnu dot org 2008-07-08 16:11
---
Subject: Bug 23455
Author: dberlin
Date: Tue Jul 8 16:11:06 2008
New Revision: 137631
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=137631
Log:
2008-07-05 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #2 from dberlin at gcc dot gnu dot org 2008-06-29 15:23 ---
You can't remove the assert, the assert is there because things will be broken
if it hasn't been handed off to do_structure_copy.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3
--- Comment #10 from dberlin at gcc dot gnu dot org 2008-06-12 14:52
---
FWIW, the comment right above the assert has proven to be true.
In a few years and releases, this is only the second time anyone has ever hit
it :)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36508
--- Comment #29 from dberlin at gcc dot gnu dot org 2008-02-11 16:29
---
Vectorization is not magic.
I'm also not sure where you got the idea that vectorization = magic speedup
There is no real expected performance gain on memory bound applications
because the processor spends all
--- Comment #3 from dberlin at gcc dot gnu dot org 2008-01-03 18:16 ---
It is never okay to base an existing expression on an SSA_NAME alone, which is
why we avoid it.
If the SSA_NAME was available, it would have been in the AVAIL set and been
found by the find part
--- Comment #11 from dberlin at gcc dot gnu dot org 2008-01-02 22:50
---
adding a comment to test
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #48 from dberlin at gcc dot gnu dot org 2007-09-11 19:55
---
4.3 is no longer a regression as it does PTA faster than 4.1, and uses less
memory.
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-08-31 19:35 ---
This one is mine.
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #27 from dberlin at gcc dot gnu dot org 2007-08-30 14:56
---
(In reply to comment #23)
I don't think the patch fixes anything.
Uh, sure it does.
Before we were ignoring the pointer results from calls.
They should point to anything.
Can you elaborate on
D.8892_26
--- Comment #5 from dberlin at gcc dot gnu dot org 2007-08-27 14:18 ---
Subject: Bug 33173
Author: dberlin
Date: Mon Aug 27 14:18:36 2007
New Revision: 127834
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=127834
Log:
2007-08-27 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #6 from dberlin at gcc dot gnu dot org 2007-08-27 19:02 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #11 from dberlin at gcc dot gnu dot org 2007-08-23 13:45
---
It did not fail for me at the point i committed the patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32772
--- Comment #1 from dberlin at gcc dot gnu dot org 2007-08-23 13:48 ---
I'm not sure what you are talking about. Do you mean nodekindf?
Do you have a case i can compile on another platform?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33159
--- Comment #13 from dberlin at gcc dot gnu dot org 2007-08-22 15:52
---
At least for 4.3, ipa-type-escape is not looking into phi_nodes for address
taking, so we end up returning false for may_alias_p (p, s) because we believe
nobody ever takes the address of s.
IE
--- Comment #15 from dberlin at gcc dot gnu dot org 2007-08-22 20:10
---
(In reply to comment #14)
--- ipa-type-escape.c.jj13 2007-08-13 15:11:18.0 +0200
+++ ipa-type-escape.c 2007-08-22 19:21:07.0 +0200
@@ -1704,6 +1704,21 @@ analyze_function (struct
--- Comment #8 from dberlin at gcc dot gnu dot org 2007-08-19 23:23 ---
Subject: Bug 32772
Author: dberlin
Date: Sun Aug 19 23:23:29 2007
New Revision: 127629
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=127629
Log:
2007-08-19 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-08-19 23:23 ---
Subject: Bug 32303
Author: dberlin
Date: Sun Aug 19 23:23:29 2007
New Revision: 127629
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=127629
Log:
2007-08-19 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #21 from dberlin at gcc dot gnu dot org 2007-08-19 23:23
---
Subject: Bug 32328
Author: dberlin
Date: Sun Aug 19 23:23:29 2007
New Revision: 127629
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=127629
Log:
2007-08-19 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #8 from dberlin at gcc dot gnu dot org 2007-08-19 23:23 ---
Subject: Bug 32716
Author: dberlin
Date: Sun Aug 19 23:23:29 2007
New Revision: 127629
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=127629
Log:
2007-08-19 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #22 from dberlin at gcc dot gnu dot org 2007-08-20 01:54
---
All should be fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-08-20 01:54 ---
All should be fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from dberlin at gcc dot gnu dot org 2007-08-20 01:54 ---
All should be fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from dberlin at gcc dot gnu dot org 2007-08-20 01:54 ---
All should be fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from dberlin at gcc dot gnu dot org 2007-08-20 01:56
---
Uh, it doesn't take 1 gig on either 4.2 or 4.3
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32723
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-08-13 12:06 ---
It takes 2 iterations, but it has come up with 144 thousand expressions
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-08-10 01:37 ---
This should be fixed now.
I believe echristo bootstrapped with this flag just yesterday, in fact.
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from dberlin at gcc dot gnu dot org 2007-08-10 01:39 ---
This should be fixed now (it was a VN/PRE interaction)
I believe echristo bootstrapped with this flag just yesterday, in fact.
The testcase is certainly not crashing for me anymore
Please reopen if it is still
--- Comment #43 from dberlin at gcc dot gnu dot org 2007-07-26 18:03
---
On my current branch, which i will commit soon, i have
tree PTA : 14.56 ( 1%) usr 0.57 ( 1%) sys 16.98 ( 1%) wall
26372 kB ( 2%) ggc
tree alias analysis : 577.90 (26%) usr 8.72 ( 8%) sys
--- Comment #7 from dberlin at gcc dot gnu dot org 2007-07-24 07:16 ---
Didn't you commit the shared bitmap fix?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32723
--- Comment #5 from dberlin at gcc dot gnu dot org 2007-07-12 19:15 ---
I have a few upcoming patches that should seriously reduce memory usage of
points-to.
If they work on mainline, i will backport them to 4.2
Please be patient, sorry for this :)
--
http://gcc.gnu.org/bugzilla
--- Comment #12 from dberlin at gcc dot gnu dot org 2007-07-12 02:20
---
Subject: Bug 32663
Author: dberlin
Date: Thu Jul 12 02:20:04 2007
New Revision: 126568
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126568
Log:
2007-07-11 Daniel Berlin [EMAIL PROTECTED]
PR
--- Comment #9 from dberlin at gcc dot gnu dot org 2007-07-07 20:09 ---
(In reply to comment #8)
I am testing this patch:
--- gcc/tree-ssa-pre.c.bad 2007-07-07 08:18:31.0 -0700
+++ gcc/tree-ssa-pre.c 2007-07-07 12:48:47.0 -0700
@@ -3362,7 +3362,8
--- Comment #22 from dberlin at gcc dot gnu dot org 2007-07-07 22:23
---
Subject: Bug 23488
Author: dberlin
Date: Sat Jul 7 22:23:26 2007
New Revision: 126449
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126449
Log:
2007-07-07 Daniel Berlin [EMAIL PROTECTED]
Revert
--- Comment #21 from dberlin at gcc dot gnu dot org 2007-07-07 03:25
---
Subject: Bug 23488
Author: dberlin
Date: Sat Jul 7 03:25:29 2007
New Revision: 126434
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126434
Log:
2007-07-06 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #7 from dberlin at gcc dot gnu dot org 2007-07-04 22:11 ---
Subject: Bug 32604
Author: dberlin
Date: Wed Jul 4 22:11:14 2007
New Revision: 126338
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126338
Log:
2007-07-04 Daniel Berlin [EMAIL PROTECTED]
PR tree
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-07-04 22:11 ---
Subject: Bug 32606
Author: dberlin
Date: Wed Jul 4 22:11:14 2007
New Revision: 126338
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126338
Log:
2007-07-04 Daniel Berlin [EMAIL PROTECTED]
PR tree
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-07-02 12:47 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-07-02 18:27 ---
Subject: Bug 32583
Author: dberlin
Date: Mon Jul 2 18:27:46 2007
New Revision: 126222
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126222
Log:
2007-07-02 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-07-02 18:27 ---
Subject: Bug 32584
Author: dberlin
Date: Mon Jul 2 18:27:46 2007
New Revision: 126222
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126222
Log:
2007-07-02 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-07-02 18:32 ---
fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-07-02 18:33 ---
fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #1 from dberlin at gcc dot gnu dot org 2007-07-02 19:40 ---
I cannot reproduce with a cross to hppa-linux.
Is this still happening with the latest patches as of this morning?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32591
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-07-02 04:35 ---
Subject: Bug 32571
Author: dberlin
Date: Mon Jul 2 04:35:37 2007
New Revision: 126186
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126186
Log:
2007-07-01 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #5 from dberlin at gcc dot gnu dot org 2007-06-30 14:15 ---
Subject: Bug 31651
Author: dberlin
Date: Sat Jun 30 14:15:26 2007
New Revision: 126149
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126149
Log:
2007-06-30 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-06-30 14:15 ---
Subject: Bug 32540
Author: dberlin
Date: Sat Jun 30 14:15:26 2007
New Revision: 126149
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126149
Log:
2007-06-30 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-06-30 14:16 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #6 from dberlin at gcc dot gnu dot org 2007-06-30 14:17 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #13 from dberlin at gcc dot gnu dot org 2007-06-20 20:09
---
(In reply to comment #10)
trunk has the same problem, but different constraints:
Constraints:
ANYTHING = ANYTHING
READONLY = ANYTHING
INTEGER = ANYTHING
barptr = bar1
barptr.0_1 = barptr
Points
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-06-10 22:41 ---
Complete guess:
alias.c relies not on TYPE_RESTRICT, but on DECL_BASED_ON_RESTRICT_P
I never noticed we even had such a thing :)
My guess is that loop unrolling makes new ssa names, and when they get
transformed
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-06-10 03:31 ---
It won't be fixed by sccvn, at least not initially.
SCCVN has very strict rules about what expressions it decides are worth trying
to keep around to simplify other expressions.
Right now, we only keep those
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-06-05 23:49 ---
And the memory usage before that revision was what?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32199
--- Comment #19 from dberlin at gcc dot gnu dot org 2007-05-18 14:46
---
Created an attachment (id=13576)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13576action=view)
Possible patch
The attached is a huge backport of the 4.3 solver changes.
I have only minimally tested it.
Let
--- Comment #9 from dberlin at gcc dot gnu dot org 2007-05-15 16:51 ---
Each one of thousands of temporary variables ends up with 12000 fields.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31863
--- Comment #25 from dberlin at gcc dot gnu dot org 2007-05-14 20:01
---
which should be the most interesting parts of the diff (apart from the
extra vops that prevent the DCE).
Danny? Any hints on what can go wrong here?
The first pass is obviously generating wrong answers
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-05-13 16:00 ---
How interesting. It is considering performing insertion on an expression that
ends up having to phi translate back through 1000's of other value handles.
But it should have already solved these, so i'm not sure
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-05-13 22:52 ---
Subject: Bug 31911
Author: dberlin
Date: Sun May 13 21:52:10 2007
New Revision: 124657
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=124657
Log:
2007-05-13 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #5 from dberlin at gcc dot gnu dot org 2007-05-13 22:52 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #10 from dberlin at gcc dot gnu dot org 2007-05-10 19:59
---
Ugh, so operand_equal_p claims two side-effecting component_refs are never the
same, unless they are pointer equal. For non-sideeffects ones, it checks
whether the operands are the same.
This causes PRE
Component: tree-optimization
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: dberlin at gcc dot gnu dot org
GCC build triplet: i686-darwin
GCC host triplet: i686-darwin
GCC target triplet: i686-darwin
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31847
--- Comment #1 from dberlin at gcc dot gnu dot org 2007-04-29 16:05 ---
PRE optimized the offset and stride variables.
SCEV doesn't know how to create an offset or step out of these optimized
operations. We used to at some point, but I believe these are peeled_chrec's
that we removed
--- Comment #33 from dberlin at gcc dot gnu dot org 2007-04-25 19:45
---
I think richi said on IRC that the following backport from 4.3 will fix it (if
so, it's the correct fix here)
Index: tree-ssa-structalias.c
--- Comment #25 from dberlin at gcc dot gnu dot org 2007-04-25 03:14
---
(In reply to comment #23)
This is a regression. Danny?
It actually should get assigned anything as a points-to set, so the bad
constraints are correct.
We should also always get correct aliasing even
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-04-16 19:14 ---
Doing cast motion actually causes about 25 *more* failures in the vectorizer
testsuite.
I'm closing this as won't fix since it seems there was no other reason to do
this.
--
dberlin at gcc dot gnu dot org
--- Comment #30 from dberlin at gcc dot gnu dot org 2007-04-17 06:53
---
Is this really still broken in mainline?
At least as of Richard's last update, it wasn't
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19580
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-04-16 02:55 ---
Don't remember if this is fixed, but not working anymore on it
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from dberlin at gcc dot gnu dot org 2007-04-16 02:56 ---
The version I produced was turned down, and I have not had time to rewrite it
to be what others wanted
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-04-16 03:12 ---
PRE actually would perform the optimization on these two cases, but it sees the
cast as an induction variable.
Found partial redundancy for expression (short unsigned int) VH.29 (VH.37)
Skipping insertion of phi
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-04-16 03:15 ---
PS, I will fix this sometime after we have LTO.
Until then, -fipa-pta is not worth it.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31068
--- Comment #1 from dberlin at gcc dot gnu dot org 2007-04-07 23:40 ---
See on-list discussion, but basically, it should suffice to simply remove the
assert, which will give the Ada frontend the same behavior it used to get.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31504
--- Comment #2 from dberlin at gcc dot gnu dot org 2007-04-08 00:50 ---
Patch was reverted
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #24 from dberlin at gcc dot gnu dot org 2007-03-17 23:07
---
Subject: Bug 29922
Author: dberlin
Date: Sat Mar 17 23:07:34 2007
New Revision: 123029
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=123029
Log:
2007-03-16 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #26 from dberlin at gcc dot gnu dot org 2007-03-17 23:18
---
Sorry for the delay.
Fixed now.
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #14 from dberlin at gcc dot gnu dot org 2007-03-14 15:12
---
(In reply to comment #13)
It's really all PTA memory.
Mainline:
tree PTA : 0.01 ( 0%) usr 0.00 ( 0%) sys 0.00 ( 0%) wall
40 kB ( 0%) ggc
TOTAL : 2.18
--- Comment #23 from dberlin at gcc dot gnu dot org 2007-03-12 19:09
---
Subject: Bug 28544
Author: dberlin
Date: Mon Mar 12 19:09:05 2007
New Revision: 122857
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=122857
Log:
2007-03-12 Daniel Berlin [EMAIL PROTECTED]
Fix PR
--- Comment #24 from dberlin at gcc dot gnu dot org 2007-03-12 19:12
---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #20 from dberlin at gcc dot gnu dot org 2007-03-09 17:55
---
Uh, i think i accidentally fixed the reduced testcase with my recent alias
patch (r122741).
Are the others still failing?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29922
--- Comment #10 from dberlin at gcc dot gnu dot org 2007-03-09 23:48
---
(In reply to comment #7)
The obvoious difference is more precise alias information:
-bar: Maximum number of VOPS needed per statement: 80
+bar: Maximum number of VOPS needed per statement: 71
See
--- Comment #15 from dberlin at gcc dot gnu dot org 2007-02-26 04:38
---
I'll happily confirm I can reproduce it on my i686-pc-linux-gnu machine
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from dberlin at gcc dot gnu dot org 2007-02-19 19:34 ---
Sorry, the fix was committed before the bug report was filed (IE ~5 minutes
before).
I forgot to mark this one closed.
--
dberlin at gcc dot gnu dot org changed:
What|Removed
--- Comment #21 from dberlin at gcc dot gnu dot org 2007-02-19 23:41
---
(In reply to comment #20)
Is there a backport of the mainline patch that I could review, or ask another
maintainer to review for inclusion in 4.2?
Thanks,
-- Mark
The attached patch named
--- Comment #11 from dberlin at gcc dot gnu dot org 2007-02-12 16:37
---
(In reply to comment #10)
Daniel, any idea?
None.
This change actually made us more conservative with points-to, it certainly
won't cause *more* things to be optimized away.
--
http://gcc.gnu.org/bugzilla
--- Comment #8 from dberlin at gcc dot gnu dot org 2007-02-12 20:47 ---
Does this work on mainline with no real issue?
If so, i'll try to backport the solver changes.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30052
--- Comment #3 from dberlin at gcc dot gnu dot org 2007-01-29 17:25 ---
Subject: Bug 30630
Author: dberlin
Date: Mon Jan 29 17:25:04 2007
New Revision: 121295
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=121295
Log:
2007-01-29 Daniel Berlin [EMAIL PROTECTED]
PR tree
--- Comment #4 from dberlin at gcc dot gnu dot org 2007-01-29 17:25 ---
Fixed
--
dberlin at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #8 from dberlin at gcc dot gnu dot org 2007-01-19 03:55 ---
Okay, well, this is pretty simple.
If we can't reproduce the bug (and i can't, and andrew can't), we can't fix it.
So this bug is just going to stay open forever until then, regardless of
whether it's a hardware
--- Comment #25 from dberlin at gcc dot gnu dot org 2006-12-29 20:28
---
So the solution for 4.3 here, from what i understand, seems pretty easy at the
tree level, no matter what reading we wish to give this.
If we add a placement_new_expr, and not try to revisit our interpretation
--- Comment #8 from dberlin at gcc dot gnu dot org 2006-12-19 14:30 ---
Is it reasonable and prudent to have tests like if (n == 0)
optimised away at the -O2 optimisation level ?
Yes
We should remember that -O2 is heavily used by lots of
customers.
And they expect a lot from
--- Comment #42 from dberlin at gcc dot gnu dot org 2006-12-11 18:13
---
This bug (against the regular development branches) should not be marked fixed
simply because a patch was applied to the redhat branch.
Either it needs to go into mainline 4.1, or the bug needs to be closed
--- Comment #4 from dberlin at gcc dot gnu dot org 2006-12-09 14:40 ---
Uh, before declaring it a reassociation issue, why don't you try turning off
reassoc and see if it actually fixes the problem.
As long as reassoc rewrites it the same everywhere, it would still be PRE'd
--- Comment #5 from dberlin at gcc dot gnu dot org 2006-12-09 15:06 ---
(In reply to comment #1)
Confirmed (but it's not PRE).
This actually is a case of PRE, because data[0] + data[3] is partially
redundant.
But according to the results, we *did* do the load PRE here.
In fact
--- Comment #4 from dberlin at gcc dot gnu dot org 2006-12-07 04:48 ---
On my machine, with an unoptimized cc1plus (IE stage1), the first one, at -O2
takes 150meg of memory total, and 221 seconds, with most of the time being
verifiers.
This is with local PTA changes to speed up PTA
--- Comment #8 from dberlin at gcc dot gnu dot org 2006-11-14 18:12 ---
Subject: Bug 27755
Author: dberlin
Date: Tue Nov 14 18:12:20 2006
New Revision: 118821
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=118821
Log:
2006-11-14 Daniel Berlin [EMAIL PROTECTED]
Fix PR
1 - 100 of 402 matches
Mail list logo
