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 C808E3858D37; Mon, 23 Oct 2023 00:28:27 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C808E3858D37
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1698020907;
	bh=2Yi5C9sw9LvvXeyZjrUiPEqe+PTHtrzmf15+l60WMtw=;
	h=From:To:Subject:Date:From;
	b=H4yA5SRtM0oSwRtnqQD6oS3Ulsde0WDX4fdLNupPP/Ef7y9uUkCHjjHH5HlMWK260
	 ivNlyfTcXZ3COoel9p59hK9/+WNhjMBfaPvuKbTuT8jaNb7wznVHc/Hp+nQabGctpH
	 PXigQKIjUQcsDbPLYQ1aKb1CBVW5X6R8uoh6qakk=
From: "pinskia at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug tree-optimization/111924] New: Look into using
 movement_possibility (in tree-ssa-loop-im.cc) for
 empty_bb_or_one_feeding_into_p for phi-opt
Date: Mon, 23 Oct 2023 00:28:27 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: new
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: tree-optimization
X-Bugzilla-Version: 14.0
X-Bugzilla-Keywords: internal-improvement
X-Bugzilla-Severity: enhancement
X-Bugzilla-Who: pinskia at gcc dot gnu.org
X-Bugzilla-Status: UNCONFIRMED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: bug_id short_desc product version bug_status
 keywords bug_severity priority component assigned_to reporter
 target_milestone
Message-ID: <bug-111924-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=3D111924

            Bug ID: 111924
           Summary: Look into using movement_possibility (in
                    tree-ssa-loop-im.cc) for
                    empty_bb_or_one_feeding_into_p for phi-opt
           Product: gcc
           Version: 14.0
            Status: UNCONFIRMED
          Keywords: internal-improvement
          Severity: enhancement
          Priority: P3
         Component: tree-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: pinskia at gcc dot gnu.org
  Target Milestone: ---

I noticed that movement_possibility/movement_possibility_1 does almost the =
same
as what some of the checks inside empty_bb_or_one_feeding_into_p does. It s=
eems
like it might be a good idea to combine the 2 into one function and use tha=
t in
both PHI-OPT and Loop-im.

Note phi-opt does allow unconditionally a few known builtins/internal funct=
ions
too which seems like could benifit IM too.
Oh movement_possibility now disallows some movements of shifts which defini=
tely
could be improved and it is definitely something which PHI-OPT should use t=
oo.=