--- Comment #20 from hjl at gcc dot gnu dot org 2009-01-30 17:32 ---
Subject: Bug 38745
Author: hjl
Date: Fri Jan 30 17:31:24 2009
New Revision: 143798
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143798
Log:
2009-01-30 H.J. Lu
2009-01-27 Richard Guenther
--- Comment #19 from rguenth at gcc dot gnu dot org 2009-01-26 15:03
---
Subject: Bug 38745
Author: rguenth
Date: Mon Jan 26 15:03:30 2009
New Revision: 143676
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143676
Log:
2009-01-26 Richard Guenther
PR tree-optimizatio
--- Comment #17 from rguenth at gcc dot gnu dot org 2009-01-26 09:55
---
Subject: Bug 38745
Author: rguenth
Date: Mon Jan 26 09:55:30 2009
New Revision: 143673
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143673
Log:
2009-01-26 Richard Guenther
PR tree-optimizatio
--- Comment #17 from rguenth at gcc dot gnu dot org 2009-01-26 09:55
---
Subject: Bug 38745
Author: rguenth
Date: Mon Jan 26 09:55:30 2009
New Revision: 143673
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143673
Log:
2009-01-26 Richard Guenther
PR tree-optimizatio
--- Comment #16 from rguenth at gcc dot gnu dot org 2009-01-25 11:40
---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|pinskia
--- Comment #15 from rguenth at gcc dot gnu dot org 2009-01-22 23:23
---
Hm? Non-TREE_ADDRESSABLE non-SSA vars are fine. It isn't DECL_GIMPLE_REG_P
either of course.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38745
--- Comment #14 from rguenth at gcc dot gnu dot org 2009-01-22 23:21
---
non-optimal tree-ssa-alias.c part:
Index: tree-ssa-alias.c
===
--- tree-ssa-alias.c(revision 143555)
+++ tree-ssa-alias.c(working copy)
@@ -2
--- Comment #13 from pinskia at gcc dot gnu dot org 2009-01-22 23:20
---
>Maybe I should merge the split of addressables from tree-ssa-alias.c from
the alias-improvements branch.
There are two things, first c64 really should not be TREE_ADDRESSABLE but
rather !DECL_GIMPLE_REG_P. But t
--- Comment #12 from rguenth at gcc dot gnu dot org 2009-01-22 23:10
---
Hm, the first forwprop dump looks sane:
VIEW_CONVERT_EXPR(c64).u.lo32.u.lo16.u.hi8 ={v} 1;
but then addressables1 comes along and rewrites c64 into SSA form ... and if
you fix it then the duplicate in tree-ssa-
--- Comment #11 from pinskia at gcc dot gnu dot org 2009-01-16 22:11
---
Semi small testcase:
union u_u16
{
unsigned short v;
struct
{
unsigned char lo8, hi8;
} __attribute__ ((__may_alias__)) u;
} __attribute__ ((__may_alias__));
union u_u32
{
unsigned int v;
struct
{
--- Comment #10 from rguenth at gcc dot gnu dot org 2009-01-16 09:34
---
Do we have a small testcase?
VIEW_CONVERT_EXPR(c64_76).u.lo32.u.lo16.u.lo8 = D.1947_8;
indeed looks funny (it's a partial def of a register).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38745
--- Comment #9 from pinskia at gcc dot gnu dot org 2009-01-16 02:11 ---
Index: tree-ssa-forwprop.c
===
--- tree-ssa-forwprop.c (revision 143415)
+++ tree-ssa-forwprop.c (working copy)
@@ -775,29 +775,43 @@ forward_propagate_
--- Comment #8 from pinskia at gcc dot gnu dot org 2009-01-16 02:01 ---
So the problem here is that we should not forward prop *(struct a*)&f into an
expression which is taking the address ...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38745
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Priority|P3 |P1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38745
--- Comment #7 from pinskia at gcc dot gnu dot org 2009-01-06 19:38 ---
(In reply to comment #6)
> It is caused by revision 141606:
>
> http://gcc.gnu.org/ml/gcc-patches/2008-11/msg00043.html
Caused, not really, exposed is more likely as I said it is really caused by the
introduction o
--- Comment #6 from hjl dot tools at gmail dot com 2009-01-06 19:33 ---
It is caused by revision 141606:
http://gcc.gnu.org/ml/gcc-patches/2008-11/msg00043.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
--- Comment #5 from pinskia at gcc dot gnu dot org 2009-01-06 18:13 ---
Note it is unrelated to PR 37553 and most likely can be reproduce with C code
...
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #4 from pinskia at gcc dot gnu dot org 2009-01-06 18:12 ---
Yes this is caused by my patch,
basically we need to stop frowproping into ADDR_EXPR, VIEW_CONVERT_EXPR.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
---
18 matches
Mail list logo
