From patchwork Thu Aug 6 01:42:06 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thomas Preud'homme X-Patchwork-Id: 504454 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 649A11402AB for ; Thu, 6 Aug 2015 11:42:32 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b=yYBXQD1W; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:references:in-reply-to:subject:date:message-id:mime-version :content-type:content-transfer-encoding; q=dns; s=default; b=EZV fzLLi5x8YIzO52J1h/nHSY6gvQEtOa5TzZXY5wPJs6F/9WOJFaOrbq1QsQ3p7aXX 9Vv0C29SsxZUXiCZV9p8emtXJ2xvldJDZcNBaUbqgpqurvrZ3qgXCOqDJdBYmFbx ++Yh1pvoUV+iAusrU7Rz2NVxqntucL5D5jYZBcWQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:references:in-reply-to:subject:date:message-id:mime-version :content-type:content-transfer-encoding; s=default; bh=aR/f9Idse Zb/o5cZd1K1msxW3D8=; b=yYBXQD1Wgc6bRIMuxsCXTAtk+os19leS5JPGiWfiq O7BILQOp7VHuWMRqK8VQHxduaEaAWBTMcVcBCnNhFebNcOEZadkBUFNHjyTBfI0/ +xvgbk5p7HEchtIL/XZ/VipjqVB/tTATpUmqg969YRi3cNOHv7VXjohfA1j3VzQc Y4= Received: (qmail 34343 invoked by alias); 6 Aug 2015 01:42:25 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 34327 invoked by uid 89); 6 Aug 2015 01:42:24 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL, BAYES_00, SPF_PASS autolearn=ham version=3.3.2 X-HELO: eu-smtp-delivery-143.mimecast.com Received: from eu-smtp-delivery-143.mimecast.com (HELO eu-smtp-delivery-143.mimecast.com) (207.82.80.143) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 06 Aug 2015 01:42:23 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) by eu-smtp-1.mimecast.com with ESMTP id uk-mta-18-qsRe0oB2QquCy-PUsCe6jg-1; Thu, 06 Aug 2015 02:42:18 +0100 Received: from SHAWIN202 ([10.1.2.79]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Thu, 6 Aug 2015 02:42:17 +0100 From: "Thomas Preud'homme" To: "'Richard Biener'" , "Jakub Jelinek" , References: <000f01d0c902$58c35be0$0a4a13a0$@arm.com> <001001d0c903$91a41980$b4ec4c80$@arm.com> In-Reply-To: <001001d0c903$91a41980$b4ec4c80$@arm.com> Subject: RE: [PATCH] Obvious fix for PR66828: left shift with undefined behavior in bswap pass Date: Thu, 6 Aug 2015 09:42:06 +0800 Message-ID: <000601d0cfe9$1c051890$540f49b0$@arm.com> MIME-Version: 1.0 X-MC-Unique: qsRe0oB2QquCy-PUsCe6jg-1 X-IsSubscribed: yes Hi, > From: gcc-patches-owner@gcc.gnu.org [mailto:gcc-patches- > owner@gcc.gnu.org] On Behalf Of Thomas Preud'homme > Sent: Tuesday, July 28, 2015 3:04 PM > > > From: gcc-patches-owner@gcc.gnu.org [mailto:gcc-patches- > > owner@gcc.gnu.org] On Behalf Of Thomas Preud'homme > > > > ChangeLog entry is as follows: > > > > 2015-07-28 Thomas Preud'homme > > > > PR tree-optimization/66828 > > * tree-ssa-math-opts.c (perform_symbolic_merge): Change type > of > > inc > > from int64_t to uint64_t. Can I backport this change to GCC 5 branch? The patch applies cleanly on GCC 5 and shows no regression on a native x86_64-linux-gnu bootstrapped GCC and an arm-none-eabi GCC cross-compiler. Best regards, Thomas diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ba37d96..a301c23 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2015-08-04 Thomas Preud'homme + + Backport from mainline + 2015-07-28 Thomas Preud'homme + + PR tree-optimization/66828 + * tree-ssa-math-opts.c (perform_symbolic_merge): Change type of inc + from int64_t to uint64_t. + 2015-08-03 John David Anglin PR target/67060 diff --git a/gcc/tree-ssa-math-opts.c b/gcc/tree-ssa-math-opts.c index c22a677..c699dcadb 100644 --- a/gcc/tree-ssa-math-opts.c +++ b/gcc/tree-ssa-math-opts.c @@ -1856,7 +1856,7 @@ perform_symbolic_merge (gimple source_stmt1, struct symbolic_number *n1, the same base (array, structure, ...). */ if (gimple_assign_rhs1 (source_stmt1) != gimple_assign_rhs1 (source_stmt2)) { - int64_t inc; + uint64_t inc; HOST_WIDE_INT start_sub, end_sub, end1, end2, end; struct symbolic_number *toinc_n_ptr, *n_end;