[Bug testsuite/104960] New: [12 regression] several test cases fail after r12-7670-gf6fb661ea8ac7e

[Bug testsuite/104960] New: [12 regression] several test cases fail after r12-7670-gf6fb661ea8ac7e

[seurer at gcc dot gnu.org]

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

            Bug ID: 104960
           Summary: [12 regression] several test cases fail after
                    r12-7670-gf6fb661ea8ac7e
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

g:f6fb661ea8ac7e17c6924719de6219f002c4efef, r12-7670-gf6fb661ea8ac7e

Maybe these test cases need updating?

FAIL: g++.dg/tree-ssa/pr86544.C  -std=gnu++14  scan-tree-dump-times phiopt4
"if" 0
FAIL: g++.dg/tree-ssa/pr86544.C  -std=gnu++17  scan-tree-dump-times phiopt4
"if" 0
FAIL: g++.dg/tree-ssa/pr86544.C  -std=gnu++20  scan-tree-dump-times phiopt4
"if" 0
FAIL: g++.dg/tree-ssa/pr86544.C  -std=gnu++98  scan-tree-dump-times phiopt4
"if" 0
FAIL: gcc.dg/tree-ssa/phi-opt-21.c scan-tree-dump phiopt4 "converted to
straightline code"
FAIL: gcc.dg/tree-ssa/popcount3.c scan-tree-dump-times phiopt4 "if" 0
FAIL: gcc.dg/tree-ssa/pr18134.c scan-tree-dump-times optimized "= a_..D. != 0"
1
FAIL: gcc.dg/tree-ssa/pr96480.c scan-tree-dump optimized " = _[0-9]* <= 3;"
FAIL: gcc.dg/tree-ssa/sccp-2.c scan-tree-dump-times optimized "bb" 1
FAIL: gcc.dg/tree-ssa/ssa-hoist-4.c scan-tree-dump-times optimized "MAX_EXPR" 1


commit f6fb661ea8ac7e17c6924719de6219f002c4efef (HEAD, refs/bisect/bad)
Author: Richard Biener <rguenther@suse.de>
Date:   Wed Mar 16 13:39:31 2022 +0100

    tree-optimization/102008 - restore if-conversion of adjacent loads

[Bug testsuite/104960] [12 regression] several test cases fail after r12-7670-gf6fb661ea8ac7e

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2022-03-17
             Status|UNCONFIRMED                 |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org
   Target Milestone|---                         |12.0

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Something went wrong with testing, will have a look.

[Bug testsuite/104960] [12 regression] several test cases fail after r12-7670-gf6fb661ea8ac7e

[cvs-commit at gcc dot gnu.org]

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

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:3a7ba8fd0cda387809e4902328af2473662b6a4a

commit r12-7687-g3a7ba8fd0cda387809e4902328af2473662b6a4a
Author: Richard Biener <rguenther@suse.de>
Date:   Thu Mar 17 08:10:59 2022 +0100

    tree-optimization/104960 - unsplit edges after late sinking

    Something went wrong when testing the earlier patch to move the
    late sinking to before the late phiopt for PR102008.  The following
    makes sure to unsplit edges after the late sinking since the split
    edges confuse the following phiopt leading to missed optimizations.

    I've went for a new pass parameter for this to avoid changing the
    CFG after the early sinking pass at this point.

    2022-03-17  Richard Biener  <rguenther@suse.de>

            PR tree-optimization/104960
            * passes.def: Add pass parameter to pass_sink_code, mark
            last one to unsplit edges.
            * tree-ssa-sink.cc (pass_sink_code::set_pass_param): New.
            (pass_sink_code::execute): Always execute TODO_cleanup_cfg
            when we need to unsplit edges.

            * gcc.dg/gimplefe-37.c: Adjust to allow either the true
            or false edge to have a forwarder.

[Bug testsuite/104960] [12 regression] several test cases fail after r12-7670-gf6fb661ea8ac7e

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.