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 E76243857BA6; Fri, 23 Sep 2022 07:48:04 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E76243857BA6
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1663919284;
	bh=KZPHtQjLJ25GvybRF5++nFBAMVcHOo6Js3qTi4OZCR4=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=GAz7RSZk2RrGCetxru/74hHlebUGNvf8mbUnhGJJJBvKkDZ8Cufvb5WCnZB53Q4Co
	 8xmBRms1e1cWr5JJwXmT997F3U6fFmmkG4z9xIxrXCzUDTlCvetPEHcJwv4CzJmTsQ
	 ZSeyNTLvMw6jR2E/TVxbaq90os6SOYXvUXzmQiQo=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug tree-optimization/106922] [12 Regression] Bogus uninitialized
 warning on boost::optional<<std::vector<std::string>>>, missed FRE
Date: Fri, 23 Sep 2022 07:48:02 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: tree-optimization
X-Bugzilla-Version: 12.2.1
X-Bugzilla-Keywords: diagnostic, 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: P2
X-Bugzilla-Assigned-To: rguenth at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 12.3
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-106922-4-2meF25wpDv@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-106922-4@http.gcc.gnu.org/bugzilla/>
References: <bug-106922-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
List-Id: <gcc-bugs.sourceware.org>

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

--- Comment #17 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jakub Jelinek <jakub@gcc.gnu.org>:

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

commit r13-2806-ga0de11d0d22054b6fd76a0730a3ec807542379d0
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Fri Sep 23 09:46:59 2022 +0200

    testsuite: Fix up pr106922.C test

    On Thu, Sep 22, 2022 at 01:10:08PM +0200, Richard Biener via Gcc-patches
wrote:
    >       * g++.dg/tree-ssa/pr106922.C: Adjust.

    > --- a/gcc/testsuite/g++.dg/tree-ssa/pr106922.C
    > +++ b/gcc/testsuite/g++.dg/tree-ssa/pr106922.C
    > @@ -87,5 +87,4 @@ void testfunctionfoo() {
    >    }
    >  }
    >
    > -// { dg-final { scan-tree-dump-times "Found fully redundant value" 4
"pre" { xfail { ! lp64 } } } }
    > -// { dg-final { scan-tree-dump-not "m_initialized" "cddce3" { xfail =
{ !
lp64 } } } }
    > +// { dg-final { scan-tree-dump-not "m_initialized" "dce3" } }

    I've noticed
    +UNRESOLVED: g++.dg/tree-ssa/pr106922.C  -std=3Dgnu++20  scan-tree-dump=
-not
dce3 "m_initialized"
    +UNRESOLVED: g++.dg/tree-ssa/pr106922.C  -std=3Dgnu++2b  scan-tree-dump=
-not
dce3 "m_initialized"
    with this change, both on x86_64 and i686.
    The dump is still cddce3, additionally as the last reference to the pre
    dump is gone, not sure it is worth creating that dump.

    With the following patch, there aren't FAILs nor UNRESOLVED tests with
    GXX_TESTSUITE_STDS=3D98,11,14,17,20,2b make check-g++
RUNTESTFLAGS=3D"--target_board=3Dunix\{-m32,-m64\} dg.exp=3D'pr106922.C'"

    2022-09-23  Jakub Jelinek  <jakub@redhat.com>

            PR tree-optimization/106922
            * g++.dg/tree-ssa/pr106922.C: Scan in cddce3 dump rather than
            dce3.  Remove -fdump-tree-pre-details from dg-options.=