From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1890 invoked by alias); 13 Aug 2011 14:28:37 -0000 Received: (qmail 1879 invoked by uid 22791); 13 Aug 2011 14:28:36 -0000 X-SWARE-Spam-Status: No, hits=-2.9 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; Sat, 13 Aug 2011 14:28:23 +0000 From: "hjl.tools at gmail dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug middle-end/50066] [4.7 Regression] Bad signed int to unsigned long long conversion Date: Sat, 13 Aug 2011 14:54: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: hjl.tools at gmail dot com X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: 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 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-08/txt/msg01244.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50066 --- Comment #5 from H.J. Lu 2011-08-13 14:27:58 UTC --- Does this patch --- diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index df7a9a2..f5e0a30 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -2065,6 +2065,12 @@ vrp_int_const_binop (enum tree_code code, tree val1, tree val2) && is_overflow_infinity (val2)) return NULL_TREE; + /* Punt integer subtraction with overflow on -MIN. */ + if (code == MINUS_EXPR + && INTEGRAL_TYPE_P (TREE_TYPE (res)) + && (sgn1 < 0 || sgn2 < 0)) + return NULL_TREE; + /* Don't try to handle division or shifting of infinities. */ if ((code == TRUNC_DIV_EXPR || code == FLOOR_DIV_EXPR --- make any senses?