[Bug tree-optimization/116922] [12 Regression] ICE: in op_iter_init, at ssa-iterators.h:604 with -Ofast

2025-04-16 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116922

Andrew Pinski  changed:

   What|Removed |Added

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

--- Comment #11 from Andrew Pinski  ---
Fixed also for GCC 12.5.0

[Bug tree-optimization/116922] [12 Regression] ICE: in op_iter_init, at ssa-iterators.h:604 with -Ofast

2025-04-16 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116922

--- Comment #10 from GCC Commits  ---
The releases/gcc-12 branch has been updated by Andrew Pinski
:

https://gcc.gnu.org/g:a5929ef809558a3117e7a6e2e66316a828b50f29

commit r12-11048-ga5929ef809558a3117e7a6e2e66316a828b50f29
Author: Andrew Pinski 
Date:   Tue Oct 1 14:48:19 2024 -0700

backprop: Fix deleting of a phi node [PR116922]

The problem here is remove_unused_var is called on a name that is
defined by a phi node but it deletes it like removing a normal statement.
remove_phi_node should be called rather than gsi_remove for phinodes.

Note there is a possibility of using simple_dce_from_worklist instead
but that is for another day.

Bootstrapped and tested on x86_64-linux-gnu.

PR tree-optimization/116922

gcc/ChangeLog:

* gimple-ssa-backprop.cc (remove_unused_var): Handle phi
nodes correctly.

gcc/testsuite/ChangeLog:

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

Signed-off-by: Andrew Pinski 
(cherry picked from commit cea87c84eacdb422caeada734ba5138c994d7022)

[Bug tree-optimization/116922] [12 Regression] ICE: in op_iter_init, at ssa-iterators.h:604 with -Ofast

2025-04-14 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116922

Andrew Pinski  changed:

   What|Removed |Added

  Known to work||13.3.1
Summary|[12/13 Regression] ICE: in  |[12 Regression] ICE: in
   |op_iter_init, at|op_iter_init, at
   |ssa-iterators.h:604 with|ssa-iterators.h:604 with
   |-Ofast  |-Ofast

--- Comment #9 from Andrew Pinski  ---
Also fixed for GCC 13.4.0.