From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 722C93890421; Tue, 16 Jun 2020 18:59:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 722C93890421 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1592333978; bh=BR43nGaEAaSMdDB3ivxwwMmNSvPe4Q/DOnn8fniAasA=; h=From:To:Subject:Date:In-Reply-To:References:From; b=KtLWNz0Ey6ZrkV2v9w9qxvsPkT9HrAA1L1ppn0PhBXV2XmKGAzD3SsznWiG2SK6Xd wKlGf6qFerptcncX5alTOTl2moSY5xWuTufTclO0ySw+qoRdY3Z54OrPz2mi86v2T/ xm6shgIb1872XWTjfA7kv59wsZ0SnfaroP6KVQZc= From: "aldyh at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/95649] [11 Regression] ICE during GIMPLE pass: cunroll since r11-1146-g1396fa5b91cfa0b3 Date: Tue, 16 Jun 2020 18:59:38 +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: 11.0 X-Bugzilla-Keywords: ice-on-valid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: aldyh at gcc dot gnu.org X-Bugzilla-Status: RESOLVED X-Bugzilla-Resolution: FIXED X-Bugzilla-Priority: P1 X-Bugzilla-Assigned-To: aldyh at gcc dot gnu.org X-Bugzilla-Target-Milestone: 11.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: resolution bug_status Message-ID: In-Reply-To: References: 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Jun 2020 18:59:38 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D95649 Aldy Hernandez changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|ASSIGNED |RESOLVED --- Comment #10 from Aldy Hernandez --- (In reply to Jeffrey A. Law from comment #8) > I still don't understand why propagating one SSA_NAME for another is caus= ing > headaches later though.=20 TBH, I don't either. When I compile the .c test in this PR with the following options: -O2 -fno-tree-scev-cprop -fdisable-tree-ccp1 -fdisable-tree-ccp2 -fdisable-tree-copyprop1 -fdisable-tree-evrp -fdisable-tree-vrp1 -fdisable-tree-ccp1 -fdisable-tree-ccp2 -fdisable-tree-ccp3 ...I still get an ICE, but there is no change in the IL between trunk witho= ut the offending patch, and with the patch that broke the test. This would indicate that there is some on-the-side structure that is being altered by = the phi argument propagation. I did notice that the IL *does* change in the middle of one of the copyprop= (or ccp?) passes, but it gets cleaned up to whatever was there before. Later in SCEV, we ICE while trying to look at the SSA_NAME_DEF_STMT of an S= SA which no longer exists in the IL. This happens in chrec_contains_symbols_defined_in_loop(). A wild guess is that the loop in= fo is getting mucked up somehow. I can investigate more deeply if desired, but since this was a silly typo, = I'm gonna leave it alone for now. Closing as fixed in mainline.=