From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugzilla@gcc.gnu.org>
Received: by sourceware.org (Postfix, from userid 48)
 id 48C413857348; Wed, 20 Apr 2022 11:28:18 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 48C413857348
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/104912] [12 Regression] 416.gamess regression after
 r12-7612-g69619acd8d9b58
Date: Wed, 20 Apr 2022 11:28:18 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: target
X-Bugzilla-Version: 12.0
X-Bugzilla-Keywords: missed-optimization
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: rguenth at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 12.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-104912-4-ibwqMgw9Iq@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-104912-4@http.gcc.gnu.org/bugzilla/>
References: <bug-104912-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-BeenThere: gcc-bugs@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-bugs mailing list <gcc-bugs.gcc.gnu.org>
List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-request@gcc.gnu.org?subject=help>
List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Wed, 20 Apr 2022 11:28:18 -0000

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

--- Comment #10 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:4d4eaa77a7d27bf7b220c3be79615410c873fcf1

commit r12-8205-g4d4eaa77a7d27bf7b220c3be79615410c873fcf1
Author: Richard Biener <rguenther@suse.de>
Date:   Mon Mar 21 14:08:25 2022 +0100

    tree-optimization/104912 - ensure cost model is checked first

    The following makes sure that when we build the versioning condition
    for vectorization including the cost model check, we check for the
    cost model and branch over other versioning checks.  That is what
    the cost modeling assumes, since the cost model check is the only
    one accounted for in the scalar outside cost.  Currently we emit
    all checks as straight-line code combined with bitwise ops which
    can result in surprising ordering of checks in the final assembly.

    Since loop_version accepts only a single versioning condition
    the splitting is done after the fact.

    The result is a 1.5% speedup of 416.gamess on x86_64 when compiling
    with -Ofast and tuning for generic or skylake.  That's not enough
    to recover from the slowdown when vectorizing but it now cuts off
    the expensive alias versioning test.

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

            PR tree-optimization/104912
            * tree-vect-loop-manip.cc (vect_loop_versioning): Split
            the cost model check to a separate BB to make sure it is
            checked first and not combined with other version checks.=