From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-293838-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 28871 invoked by alias); 7 Jun 2011 14:12:27 -0000
Received: (qmail 28785 invoked by uid 22791); 7 Jun 2011 14:12:27 -0000
X-SWARE-Spam-Status: No, hits=-2.2 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,RFC_ABUSE_POST
X-Spam-Check-By: sourceware.org
Received: from mail-pz0-f47.google.com (HELO mail-pz0-f47.google.com) (209.85.210.47)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 07 Jun 2011 14:12:13 +0000
Received: by pzk36 with SMTP id 36so2651314pzk.20        for <gcc-patches@gcc.gnu.org>; Tue, 07 Jun 2011 07:12:13 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.68.2.1 with SMTP id 1mr231593pbq.102.1307455932841; Tue, 07 Jun 2011 07:12:12 -0700 (PDT)
Received: by 10.142.78.4 with HTTP; Tue, 7 Jun 2011 07:12:12 -0700 (PDT)
In-Reply-To: <BANLkTimpd0f-WDpZEA_uHt2MutWN=mMDdQ@mail.gmail.com>
References: <BANLkTinDbrWcYi6x1zDsaC34M1tA1iC1oQ@mail.gmail.com>	<BANLkTin01-YvDPXskejB8TcX59VGHAMcxw@mail.gmail.com>	<BANLkTimozGYneD-6xQn7qo1LYo82pEA26Q@mail.gmail.com>	<BANLkTimpd0f-WDpZEA_uHt2MutWN=mMDdQ@mail.gmail.com>
Date: Tue, 07 Jun 2011 14:12:00 -0000
Message-ID: <BANLkTikyJb4fwE+h0xMFLNyL1K5NtSfLqg@mail.gmail.com>
Subject: Re: [PATCH, i386]: Remaining FP moves cleanups
From: Uros Bizjak <ubizjak@gmail.com>
To: "H.J. Lu" <hjl.tools@gmail.com>
Cc: gcc-patches@gcc.gnu.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
X-SW-Source: 2011-06/txt/msg00523.txt.bz2

On Tue, Jun 7, 2011 at 4:07 PM, H.J. Lu <hjl.tools@gmail.com> wrote:

>> OTOH, you are right, this statement can be moved to case 6, without
>> the check for registers.
>>
>
> Like this? =A0OK for trunk with a ChangeLog entry?
>
> Thanks.
>
> --
> H.J.
> ----
> diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
> index 58de87b..a61bffb 100644
> --- a/gcc/config/i386/i386.md
> +++ b/gcc/config/i386/i386.md
> @@ -3207,11 +3207,11 @@
> =A0 =A0 case 6:
> =A0 =A0 =A0 if (get_attr_mode (insn) =3D=3D MODE_V4SF)
> =A0 =A0 =A0 =A0return "%vmovaps\t{%1, %0|%0, %1}";
> + =A0 =A0 =A0else if (TARGET_AVX && REG_P (operands[0]) && REG_P (operand=
s[1]))

if (TARGET_AVX)

> + =A0 =A0 =A0 return "vmovss\t{%1, %0, %0|%0, %0, %1}";
>
> =A0 =A0 case 7:
> =A0 =A0 case 8:
> - =A0 =A0 =A0if (TARGET_AVX && REG_P (operands[0]) && REG_P (operands[1]))
> - =A0 =A0 =A0 return "vmovss\t{%1, %0, %0|%0, %0, %1}";
> =A0 =A0 =A0 return "%vmovss\t{%1, %0|%0, %1}";
>
> =A0 =A0 case 9:
>

OK with this change and suitable ChangeLog entry.

Thanks,
Uros.