[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

Richard Biener  changed:

   What|Removed |Added

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

--- Comment #6 from Richard Biener  ---
Fixed then.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread marxin at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

--- Comment #5 from Martin Liška  ---
(In reply to Richard Biener from comment #4)
> Fixed on trunk, not sure how important it is to backport.

I wouldn't backport it. The strange options come from my periodic fuzzing and
it took quite some time to find this issue.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

Richard Biener  changed:

   What|Removed |Added

  Known to fail|11.0|
  Known to work||11.0

--- Comment #4 from Richard Biener  ---
Fixed on trunk, not sure how important it is to backport.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

--- Comment #3 from CVS Commits  ---
The master branch has been updated by Richard Biener :

https://gcc.gnu.org/g:4dcc7f03b54087638e084ac69d40d7507fe83bd8

commit r11-3456-g4dcc7f03b54087638e084ac69d40d7507fe83bd8
Author: Richard Biener 
Date:   Fri Sep 25 13:08:48 2020 +0200

tree-optimization/97199 - fix virtual operand update in if-conversion

This fixes a corner case with virtual operand update in if-conversion
by re-organizing the code to remove edges only after the last point
we need virtual PHI operands to be available.

2020-09-25  Richard Biener  

PR tree-optimization/97199
* tree-if-conv.c (combine_blocks): Remove edges only
after looking at virtual PHI args.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

--- Comment #2 from Richard Biener  ---
(gdb) p debug_gimple_stmt (phi)
.MEM_104 = PHI <(4), .MEM_97(41)>

so clearly bogus IL here.  The issue is the virtual operand update in
if-conversion combine_blocks which does

  /* We release virtual PHIs late because we have to propagate them
 out using the current VUSE.  The def might be the one used
 after the loop.  */
  vphi = get_virtual_phi (bb);
  if (vphi)
{
  /* When there's just loads inside the loop a stray virtual
 PHI merging the uses can appear, update last_vdef from
 it.  */
  if (!last_vdef)
last_vdef = gimple_phi_arg_def (vphi, 0);

but this doesn't work since we already removed all edges which clears
the PHI arg defs.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

Richard Biener  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |rguenth at gcc dot 
gnu.org
 Status|NEW |ASSIGNED

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

Richard Biener  changed:

   What|Removed |Added

  Known to fail||10.2.1, 11.0
Version|10.0|10.2.1

--- Comment #1 from Richard Biener  ---
Confirmed, I'll have a look.

[Bug tree-optimization/97199] ICE in process_bb at gcc/tree-ssa-sccvn.c:7250

2020-09-25 Thread marxin at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97199

Martin Liška  changed:

   What|Removed |Added

 Ever confirmed|0   |1
   Last reconfirmed||2020-09-25
 Status|UNCONFIRMED |NEW