From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 122924 invoked by alias); 26 Mar 2015 13:19:02 -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 122622 invoked by uid 48); 26 Mar 2015 13:18:59 -0000 From: "mpolacek at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug sanitizer/65583] [5 Regression][UBSAN] ICE segfault in inline_edge_summary Date: Thu, 26 Mar 2015 13:38:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: sanitizer X-Bugzilla-Version: 5.0 X-Bugzilla-Keywords: ice-on-valid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: mpolacek at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 5.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2015-03/txt/msg02968.txt.bz2 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65583 --- Comment #9 from Marek Polacek --- (In reply to Jakub Jelinek from comment #8) > So, can you try to schedule another > NEXT_PASS (pass_rebuild_cgraph_edges); > right after ubsan pass if that fixes it? > And then move that right before ubsan and see if it is broken again? Yup: scheduling the pass after ubsan fixes it, scheduling is before ICEs as well. So either simply that, or something like --- a/gcc/passes.def +++ b/gcc/passes.def @@ -57,6 +57,9 @@ along with GCC; see the file COPYING3. If not see NEXT_PASS (pass_init_datastructures); NEXT_PASS (pass_build_ssa); NEXT_PASS (pass_ubsan); + PUSH_INSERT_PASSES_WITHIN (pass_ubsan) + NEXT_PASS (pass_rebuild_cgraph_edges); + POP_INSERT_PASSES () NEXT_PASS (pass_early_warn_uninitialized); POP_INSERT_PASSES () (Maybe that PUSH_INSERT_PASSES_WITHIN makes it cheaper for non-ubsan compilation?)