From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11173 invoked by alias); 30 Apr 2013 10:53:25 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Received: (qmail 11122 invoked by uid 48); 30 Apr 2013 10:53:22 -0000 From: "rguenth at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug middle-end/57107] [4.9 Regression] tree check fail in unlink_stmt_vdef Date: Tue, 30 Apr 2013 10:53:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: middle-end X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenth at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: rguenth at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.9.0 X-Bugzilla-Changed-Fields: CC Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 X-SW-Source: 2013-04/txt/msg02406.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57107 Richard Biener changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jakub at gcc dot gnu.org --- Comment #5 from Richard Biener 2013-04-30 10:53:21 UTC --- (gdb) call debug_gimple_stmt (stmt) # .MEM_159(ab) = VDEF <.MEM> D.63506 ={v} {CLOBBER}; SSA form is not up-to-date. It's a latent issue with sinking clobbers during EH cleanup. We have : # .MEM_32(ab) = VDEF <.MEM_31(ab)> D.2688 ={v} {CLOBBER}; resx 3 : # .MEM_61 = VDEF <.MEM_32(ab)> MEM[(struct new_allocator *)&D.2706] ={v} {CLOBBER}; # .MEM_62 = VDEF <.MEM_61> D.2706 ={v} {CLOBBER}; # .MEM_34 = VDEF <.MEM_62> D.2706 ={v} {CLOBBER}; resx 2 : # .MEM_35 = VDEF <.MEM_34> t ={v} {CLOBBER}; resx 1 and first sink D.2688 ={v} {CLOBBER}; into the successor (L7), dropping virtual operands. Then we continue sinking but call unlink_stmt_vdef on the moved stmt which no longer has up-to-date SSA form and thus we ICE. As we only sink across forwarder edges we can trivially keep the original virtual operands in place. I have a patch.