From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 130010 invoked by alias); 7 Sep 2016 12:59:42 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 129994 invoked by uid 89); 7 Sep 2016 12:59:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=sk:kyrylo, kyrylo, kyrylotkachovarmcom, kyrylo.tkachov@arm.com X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 07 Sep 2016 12:59:34 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BBCABF0 for ; Wed, 7 Sep 2016 05:59:32 -0700 (PDT) Received: from [10.2.207.77] (e100706-lin.cambridge.arm.com [10.2.207.77]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 6D3BE3F211 for ; Wed, 7 Sep 2016 05:59:32 -0700 (PDT) Message-ID: <57D00F32.9010801@foss.arm.com> Date: Wed, 07 Sep 2016 13:07:00 -0000 From: Kyrill Tkachov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: GCC Patches Subject: [PATCH][expmed.c] PR middle-end/77426 Delete duplicate condition in synth_mult Content-Type: multipart/mixed; boundary="------------020304020904070909070704" X-SW-Source: 2016-09/txt/msg00387.txt.bz2 This is a multi-part message in MIME format. --------------020304020904070909070704 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-length: 421 Hi all, The duplicate mode check in synth can just be deleted IMO. It was introduced as part of r139821 that was a much larger change introducing size/speed differentiation to the RTL midend. So I think it's just a typo/copy-pasto. Tested on aarch64-none-elf. Ok? Thanks, Kyrill 2016-09-07 Kyrylo Tkachov PR middle-end/77426 * expmed.c (synth_mult): Delete duplicate mode check. --------------020304020904070909070704 Content-Type: text/x-patch; name="expmed-mode.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="expmed-mode.patch" Content-length: 487 diff --git a/gcc/expmed.c b/gcc/expmed.c index 1cedf023c8e8916d887bd3a9d9a723e3cc2354f7..a5da8836f21debcda3b834cb869348ea6cb33414 100644 --- a/gcc/expmed.c +++ b/gcc/expmed.c @@ -2572,7 +2572,6 @@ synth_mult (struct algorithm *alg_out, unsigned HOST_WIDE_INT t, entry_ptr = alg_hash_entry_ptr (hash_index); if (entry_ptr->t == t && entry_ptr->mode == mode - && entry_ptr->mode == mode && entry_ptr->speed == speed && entry_ptr->alg != alg_unknown) { --------------020304020904070909070704--