[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2013-01-24 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



Martin Jambor  changed:



   What|Removed |Added



 Status|ASSIGNED|RESOLVED

 Resolution||FIXED



--- Comment #6 from Martin Jambor  2013-01-24 
15:52:56 UTC ---

Finally fixed everywhere.


[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2013-01-24 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



--- Comment #5 from Martin Jambor  2013-01-24 
15:41:19 UTC ---

Author: jamborm

Date: Thu Jan 24 15:41:04 2013

New Revision: 195429



URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195429

Log:

2013-01-24  Martin Jambor  



Backport from mainline

2013-01-04  Martin Jambor  



PR tree-optimization/55755

* tree-sra.c (sra_modify_assign): Do not check that an access has no

children when trying to avoid producing a VIEW_CONVERT_EXPR.



testsuite/

* gcc.dg/torture/pr55755.c: New test.

* gcc.dg/tree-ssa/sra-13.c: Likewise.

* gcc.dg/tree-ssa/pr45144.c: Update.





Added:

branches/gcc-4_6-branch/gcc/testsuite/gcc.dg/torture/pr55755.c

branches/gcc-4_6-branch/gcc/testsuite/gcc.dg/tree-ssa/sra-13.c

Modified:

branches/gcc-4_6-branch/gcc/ChangeLog

branches/gcc-4_6-branch/gcc/testsuite/ChangeLog

branches/gcc-4_6-branch/gcc/testsuite/gcc.dg/tree-ssa/pr45144.c

branches/gcc-4_6-branch/gcc/tree-sra.c


[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2013-01-24 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



--- Comment #4 from Martin Jambor  2013-01-24 
14:54:02 UTC ---

Author: jamborm

Date: Thu Jan 24 14:53:56 2013

New Revision: 195425



URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195425

Log:

2013-01-24  Martin Jambor  



Backport from mainline

2013-01-04  Martin Jambor  



PR tree-optimization/55755

* tree-sra.c (sra_modify_assign): Do not check that an access has no

children when trying to avoid producing a VIEW_CONVERT_EXPR.



testsuite/

* gcc.dg/torture/pr55755.c: New test.

* gcc.dg/tree-ssa/sra-13.c: Likewise.

* gcc.dg/tree-ssa/pr45144.c: Update.





Added:

branches/gcc-4_7-branch/gcc/testsuite/gcc.dg/torture/pr55755.c

branches/gcc-4_7-branch/gcc/testsuite/gcc.dg/tree-ssa/sra-13.c

Modified:

branches/gcc-4_7-branch/gcc/ChangeLog

branches/gcc-4_7-branch/gcc/testsuite/ChangeLog

branches/gcc-4_7-branch/gcc/testsuite/gcc.dg/tree-ssa/pr45144.c

branches/gcc-4_7-branch/gcc/tree-sra.c


[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2013-01-04 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



--- Comment #3 from Martin Jambor  2013-01-04 
13:20:50 UTC ---

Author: jamborm

Date: Fri Jan  4 13:20:38 2013

New Revision: 194905



URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194905

Log:

2013-01-04  Martin Jambor  



PR tree-optimization/55755

* tree-sra.c (sra_modify_assign): Do not check that an access has no

children when trying to avoid producing a VIEW_CONVERT_EXPR.



testsuite/

* gcc.dg/torture/pr55755.c: New test.

* gcc.dg/tree-ssa/sra-13.c: Likewise.

* gcc.dg/tree-ssa/pr45144.c: Update.





Added:

trunk/gcc/testsuite/gcc.dg/torture/pr55755.c

trunk/gcc/testsuite/gcc.dg/tree-ssa/sra-13.c

Modified:

trunk/gcc/ChangeLog

trunk/gcc/testsuite/ChangeLog

trunk/gcc/testsuite/gcc.dg/tree-ssa/pr45144.c

trunk/gcc/tree-sra.c


[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2013-01-04 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



Martin Jambor  changed:



   What|Removed |Added



URL||http://gcc.gnu.org/ml/gcc-p

   ||atches/2013-01/msg00132.htm

   ||l



--- Comment #2 from Martin Jambor  2013-01-04 
13:02:51 UTC ---

Patch in the mailing list:

http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00132.html


[Bug tree-optimization/55755] Invalid VIEW_CONVERT_EXPR produced by SRA

2012-12-20 Thread jamborm at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55755



Martin Jambor  changed:



   What|Removed |Added



 Status|UNCONFIRMED |ASSIGNED

   Last reconfirmed||2012-12-20

 AssignedTo|unassigned at gcc dot   |jamborm at gcc dot gnu.org

   |gnu.org |

 Ever Confirmed|0   |1



--- Comment #1 from Martin Jambor  2012-12-20 
14:28:07 UTC ---

Obviously mine.  The fix for release branches is probably going to be

add !access->grp_unscalarizable_region test to most to a few

access_has_children_p tests.  The proper fix is to re-work

access_has_children_p to a predicate returning true if there are any

replacements in any of its children.  But let me audit the

access_has_children_p tests first.