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 5EF123858D28; Fri,  7 Jan 2022 09:50:45 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 5EF123858D28
From: "tomas.kalibera at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/103465] [12 regression] -freorder-blocks-and-partition
 broken on 64-bit Windows
Date: Fri, 07 Jan 2022 09:50:45 +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: 11.2.0
X-Bugzilla-Keywords: diagnostic, patch
X-Bugzilla-Severity: major
X-Bugzilla-Who: tomas.kalibera at gmail dot com
X-Bugzilla-Status: REOPENED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: marxin at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 12.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-103465-4-0A9ClFtoP0@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-103465-4@http.gcc.gnu.org/bugzilla/>
References: <bug-103465-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: Fri, 07 Jan 2022 09:50:45 -0000

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

--- Comment #27 from Tomas Kalibera <tomas.kalibera at gmail dot com> ---

> > should do the job.  Tomas, can you give it a try?
>=20
> Thanks, so far I tried it only on predict-22.c and it works (with a fixed
> comma as below), enables the optimization. I will do more testing tomorro=
w.

I've tested with

gcc12 (6aa0859afaf28f4fb13121352225bc5877e02a44)
gcc11 (a03aae8d9f5dbfe3ca3dbfe7eadc6bbe6fbbe1bc)
gcc10 (48e0da239f65b7cfa0e6f51f266c2e04f5ad9bbd)

I confirm that with the patch applied:

* the original bug report about invalid note about
'-freorder-blocks-and-partition' is fixed

* the optimization is applied by default at -O3, -O2 and can be enabled at =
-O1
via -f

* the optimization can be disabled via -fno- and via pragma

* -Q --help=3Doptimizers issue for the option goes away, so "fixed" (though=
 as I
understand from Martin other optimizations still disabled by target will
incorrectly appear as enabled)

So with this patch applied, I am happy for this bug report to be closed. Th=
anks
to both of you.=