This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][x86] Match movss and movsd "blend" instructions
- From: Jeff Law <law at redhat dot com>
- To: Allan Sandfeld Jensen <linux at carewolf dot com>, gcc-patches at gcc dot gnu dot org, Jakub Jelinek <jakub at redhat dot com>
- Cc: Uros Bizjak <ubizjak at gmail dot com>, Marc Glisse <marc dot glisse at inria dot fr>
- Date: Tue, 14 Aug 2018 22:33:57 -0600
- Subject: Re: [PATCH][x86] Match movss and movsd "blend" instructions
- References: <6341719.0aBNjb40xM@twilight> <14233194.OQ7Imlunx6@twilight> <20180811091839.GD20910@laptop.zalov.cz> <1886122.85WXuti7Ez@twilight>
On 08/11/2018 03:54 AM, Allan Sandfeld Jensen wrote:
> On Samstag, 11. August 2018 11:18:39 CEST Jakub Jelinek wrote:
>> On Sat, Aug 11, 2018 at 10:59:26AM +0200, Allan Sandfeld Jensen wrote:
>>> +/* A subroutine of ix86_expand_vec_perm_builtin_1. Try to implement D
>>> + using movss or movsd. */
>>> +static bool
>>> +expand_vec_perm_movs (struct expand_vec_perm_d *d)
>>> +{
>>> + machine_mode vmode = d->vmode;
>>> + unsigned i, nelt = d->nelt;
>>> + rtx x;
>>> +
>>> + if (d->one_operand_p)
>>> + return false;
>>> +
>>> + if (TARGET_SSE2 && (vmode == V2DFmode || vmode == V4SFmode))
>>> + ;
>>> + else
>>> + return false;
>>> +
>>> + /* Only the first element is changed. */
>>
>> Two spaces after .
>>
>>> + if (d->perm[0] != nelt && d->perm[0] != 0)
>>> + return false;
>>> + for (i = 1; i < nelt; ++i) {
>>> + {
>>> + if (d->perm[i] != i + nelt - d->perm[0])
>>> + return false;
>>> + }
>>> + }
>>
>> Extraneous {}s (both pairs, the outer ones even badly indented).
>>
>> Otherwise LGTM.
>>
> Updated:
>
> Note as an infrequent contributor don't have commit access, so I need someone
> reviewing to also commit.
I fixed up the ChangeLog, extracted the test from the original patch and
committed all the bits to the trunk.
Thanks,
jeff