From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 77A613858421; Tue, 14 Mar 2023 08:11:04 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 77A613858421 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1678781464; bh=Ey+0xxF66xEjgxZSGw+uu3BMpswCppFxQ+0NAWu00OI=; h=From:To:Subject:Date:In-Reply-To:References:From; b=Qi4nHrVJr3iS03464xAqNWWOPgh9p1kkkBbWs60tptA2XDvD8kzYXRjSOKoA4ODxA BEXwlg1g1bFrpvZcDe0+HOksZOIwz+3QYWoRScx0/Gv6cwhMAl5zxmNUbKPm176b4F 3z1MfaYFBRoX9AG9MEGXzXaZdIVcQoiIbjy53kGk= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/106896] [13 Regression] ICE in to_sreal_scale, at profile-count.cc:339 since r13-2288-g61c4c989034548f4 Date: Tue, 14 Mar 2023 08:11: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: 13.0 X-Bugzilla-Keywords: ice-on-valid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: cvs-commit at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P1 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 13.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: 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 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D106896 --- Comment #12 from CVS Commits --- The master branch has been updated by Jan Hubicka : https://gcc.gnu.org/g:5159a1f1e91e03d4b82808a0062697318232543f commit r13-6652-g5159a1f1e91e03d4b82808a0062697318232543f Author: Jan Hubicka Date: Tue Mar 14 09:10:35 2023 +0100 Fix overactive sanity check in profile_count::to_sreal_scale As discussed in the PR log, profile_count::to_cgraph_frequency was originally intended to work across function boundary and has some extra logic and sanity check for that. It is used only within single function and with current API it can not really work well globally, so this patch synchronizes its implementation with probability_in which does similar job but to determ= ine relative probability. gcc/ChangeLog: 2023-03-14 Jan Hubicka PR tree-optimization/106896 * profile-count.cc (profile_count::to_sreal_scale): Synchronize implementatoin with probability_in; avoid some asserts.=