https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97953

--- Comment #11 from Chris Clayton <chris2553 at googlemail dot com> ---
I've finished the bisect and landed at:

[chris:~/scratch/gcc-ICE/gcc]$ git bisect good
bd87cc14ebdb6789e067fb1828d5808407c308b3 is the first bad commit
commit bd87cc14ebdb6789e067fb1828d5808407c308b3
Author: Richard Biener <rguent...@suse.de>
Date:   Wed Nov 11 11:51:59 2020 +0100

    tree-optimization/97623 - Avoid PRE hoist insertion iteration

    The recent previous change in this area limited hoist insertion
    iteration via a param but the following is IMHO better since
    we are not really interested in PRE opportunities exposed by
    hoisting but only the other way around.  So this moves hoist
    insertion after PRE iteration finished and removes hoist
    insertion iteration alltogether.

    2020-11-11  Richard Biener  <rguent...@suse.de>

            PR tree-optimization/97623
            * params.opt (-param=max-pre-hoist-insert-iterations): Remove
            again.
            * doc/invoke.texi (max-pre-hoist-insert-iterations): Likewise.
            * tree-ssa-pre.c (insert): Move hoist insertion after PRE
            insertion iteration and do not iterate it.

            * gcc.dg/tree-ssa/ssa-hoist-3.c: Adjust.
            * gcc.dg/tree-ssa/ssa-hoist-7.c: Likewise.
            * gcc.dg/tree-ssa/ssa-pre-30.c: Likewise.

 gcc/doc/invoke.texi                         |  5 -----
 gcc/params.opt                              |  4 ----
 gcc/testsuite/gcc.dg/tree-ssa/ssa-hoist-3.c |  2 +-
 gcc/testsuite/gcc.dg/tree-ssa/ssa-hoist-7.c |  4 ++--
 gcc/testsuite/gcc.dg/tree-ssa/ssa-pre-30.c  |  2 +-
 gcc/tree-ssa-pre.c                          | 34 +++++++++++++++++++----------
 6 files changed, 26 insertions(+), 25 deletions(-)

I've also confirmed the outcome. A build with this commit at HEAD fails with
the ICE. A build with the commits parent at HEAD succeeds.

I'll attach the bisect log in a few minutes.

Reply via email to