From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31759 invoked by alias); 23 May 2011 14:10:52 -0000 Received: (qmail 31734 invoked by uid 22791); 23 May 2011 14:10:51 -0000 X-SWARE-Spam-Status: No, hits=-2.7 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 23 May 2011 14:10:37 +0000 From: "jakub at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug lto/49123] [4.6 Regression] FAIL: gcc.c-torture/execute/pr48973-[12].c X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: lto X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: jakub 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: --- X-Bugzilla-Changed-Fields: Target Milestone 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 Date: Mon, 23 May 2011 14:17:00 -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 X-SW-Source: 2011-05/txt/msg02063.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49123 Jakub Jelinek changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.6.1 |--- --- Comment #6 from Jakub Jelinek 2011-05-23 13:36:00 UTC --- (In reply to comment #4) > A less intrusive variant of the patch might be suitable for backporting > (it doesn't touch the NULL type argument path) and fixes the testcase for me. > > Index: gcc/tree.c > =================================================================== > --- gcc/tree.c (revision 174066) > +++ gcc/tree.c (working copy) > @@ -1034,9 +1034,9 @@ build_int_cst (tree type, HOST_WIDE_INT > { > /* Support legacy code. */ > if (!type) > - type = integer_type_node; > + return build_int_cst_wide (integer_type_node, low, low < 0 ? -1 : 0); > > - return build_int_cst_wide (type, low, low < 0 ? -1 : 0); > + return double_int_to_tree (type, shwi_to_double_int (low)); > } > > /* Create an INT_CST node with a LOW value in TYPE. The value is sign > extended Alternatively you could call build_int_cst_type instead of build_int_cst in the spot where FRE? calls it. Guess that would be even safer to backport.