From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 100462 invoked by alias); 1 Aug 2017 12:04:27 -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 100384 invoked by uid 89); 1 Aug 2017 12:04:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:704, months X-HELO: mail-ua0-f177.google.com Received: from mail-ua0-f177.google.com (HELO mail-ua0-f177.google.com) (209.85.217.177) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 01 Aug 2017 12:04:15 +0000 Received: by mail-ua0-f177.google.com with SMTP id 80so5983165uas.0 for ; Tue, 01 Aug 2017 05:04:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=evTSZqha4mSOdkHwhYzm+YF0Gxc8oP+GhuIR2+PuJQ4=; b=QT/IlBGi0Izgmq2e5ByJJ9W3cscs7ALKHHDvZ3tZisTdc+O6P3RIfH0TVpbxF65XB3 rXcZRU6P+XnaPlutcVe6CsqXrGDwuY3TebB20skXIbz2F5cD+Jb2TFtEOFyr6e+TyjuP q+wedWxxKVJqkz5aBUULrFsXOl8+0bVGUkZzN7dZh3TtUvB5Qat8m9EwB5Q8AUoS50jU L0lbMIQb7dloCqYGt2foqcnpI6MT24Bhv4tbt7BzFDsB6cBTPv0RxrkkoFPXd9HaOFK3 /e6nmHfDUXxyRFHL4uR7LEL5eDpjmiNftuPf+AToD1ufqv9Ztw4A5m0d0dy7N3z3afum 0vYw== X-Gm-Message-State: AIVw110D/gSxMr7XhPWD1+rB1hBkWH46fzxV0d/8B47mOg/fZop+u+Pm C7UEbdZTZ0rjNTHkdkAzkkjIIAjwWg== X-Received: by 10.176.81.34 with SMTP id e31mr14024763uaa.73.1501589053446; Tue, 01 Aug 2017 05:04:13 -0700 (PDT) MIME-Version: 1.0 Received: by 10.103.34.5 with HTTP; Tue, 1 Aug 2017 05:04:12 -0700 (PDT) In-Reply-To: References: <20170614084233.GA15599@arm.com> From: "Bin.Cheng" Date: Tue, 01 Aug 2017 12:04:00 -0000 Message-ID: Subject: Re: [PATCH][GCC][AArch64] optimize float immediate moves (2 /4) - HF/DF/SF mode. To: Tamar Christina Cc: James Greenhalgh , GCC Patches , nd , Marcus Shawcroft , Richard Earnshaw Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2017-08/txt/msg00070.txt.bz2 On Tue, Aug 1, 2017 at 12:51 PM, Tamar Christina wrote: >> >> Given review comment already pointed out big-endian issue and patch was >> updated to address it, I would expect reg-test on a big-endian target before >> applying patch, right? > > The patch spent 6 months in external review. > Given that, I simply forgot to rerun big endian before the commit as I did the rest. > > The failing tests were all added after the submission of this patch. I'll have a look. I may bisect to wrong commit? The patch is committed three days ago, how could all failing tests be added after it? Thanks, bin