[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-07 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #9 from Richard Biener  ---
Author: rguenth
Date: Wed May  7 08:06:45 2014
New Revision: 210145

URL: http://gcc.gnu.org/viewcvs?rev=210145&root=gcc&view=rev
Log:
2014-05-07  Richard Biener  

PR tree-optimization/57864
* gcc.dg/torture/pr57864.c: New testcase.

Added:
branches/gcc-4_8-branch/gcc/testsuite/gcc.dg/torture/pr57864.c
Modified:
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-07 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

Richard Biener  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Richard Biener  ---
Fixed.


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-07 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #7 from Richard Biener  ---
Author: rguenth
Date: Wed May  7 08:05:57 2014
New Revision: 210144

URL: http://gcc.gnu.org/viewcvs?rev=210144&root=gcc&view=rev
Log:
2014-05-07  Richard Biener  

PR tree-optimization/57864
* tree-ssa-pre.c (phi_translate_1): Backport NAME case
simplification from mainline.  Do not lookup the VN
value-number here.

* gcc.dg/torture/pr57864.c: New testcase.

Added:
branches/gcc-4_7-branch/gcc/testsuite/gcc.dg/torture/pr57864.c
Modified:
branches/gcc-4_7-branch/gcc/ChangeLog
branches/gcc-4_7-branch/gcc/testsuite/ChangeLog
branches/gcc-4_7-branch/gcc/tree-ssa-pre.c


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-07 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #6 from Richard Biener  ---
Author: rguenth
Date: Wed May  7 08:05:17 2014
New Revision: 210143

URL: http://gcc.gnu.org/viewcvs?rev=210143&root=gcc&view=rev
Log:
2014-05-07  Richard Biener  

PR tree-optimization/57864
* gcc.dg/torture/pr57864.c: New testcase.

Added:
branches/gcc-4_9-branch/gcc/testsuite/gcc.dg/torture/pr57864.c
Modified:
branches/gcc-4_9-branch/gcc/testsuite/ChangeLog


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-07 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #5 from Richard Biener  ---
Author: rguenth
Date: Wed May  7 08:04:15 2014
New Revision: 210142

URL: http://gcc.gnu.org/viewcvs?rev=210142&root=gcc&view=rev
Log:
2014-05-07  Richard Biener  

PR tree-optimization/57864
* gcc.dg/torture/pr57864.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.dg/torture/pr57864.c
Modified:
trunk/gcc/testsuite/ChangeLog


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #4 from Richard Biener  ---
Caused by

static pre_expr
phi_translate_1 (pre_expr expr, bitmap_set_t set1, bitmap_set_t set2,
 basic_block pred, basic_block phiblock)
{
...
case NAME:
  {
...
   if (TREE_CODE (def) == SSA_NAME)
  def = VN_INFO (def)->valnum;

picking up VN inserted expressions.  I re-wrote this as part of

2012-09-24  Richard Guenther  

   * tree-ssa-pre.c (bitmap_find_leader, create_expression_by_pieces,
   find_or_generate_expression): Remove dominating stmt argument.
   (find_leader_in_sets, phi_translate_1, bitmap_find_leader,
   create_component_ref_by_pieces_1, create_component_ref_by_pieces,
   do_regular_insertion, do_partial_partial_insertion): Adjust.
   (compute_avail): Do not set uids.

and that fixed it.  I am going to try backporting that change in isolation.


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2014-05-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

Richard Biener  changed:

   What|Removed |Added

   Keywords||ice-on-valid-code
   Priority|P3  |P2
  Known to work||4.7.2, 4.8.0
  Known to fail||4.7.3


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2013-08-27 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

Richard Biener  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2013-08-27
   Assignee|unassigned at gcc dot gnu.org  |rguenth at gcc dot 
gnu.org
   Target Milestone|--- |4.7.4
 Ever confirmed|0   |1

--- Comment #3 from Richard Biener  ---
Confirmed, mine.  I _think_ I have fixed a duplicate of this on trunk/4.8 ...


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2013-07-09 Thread mikpe at it dot uu.se
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #2 from Mikael Pettersson  ---
The ICE on 4.7 branch started with the PR55107 backport in r195755.


[Bug tree-optimization/57864] [4.7 Regression] ICE in bitmap_set_replace_value, at tree-ssa-pre.c:862

2013-07-09 Thread mikpe at it dot uu.se
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57864

--- Comment #1 from Mikael Pettersson  ---
Created attachment 30486
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30486&action=edit
slightly reduced test case in plain C

Doesn't depend on C++, this plain C version also ICEs 4.7.3.