public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Christophe Lyon <christophe.lyon@linaro.org>
To: "Yangfei (Felix)" <felix.yang@huawei.com>
Cc: "ramrad01@arm.com" <ramrad01@arm.com>,
	Michael Collison <michael.collison@linaro.org>,
		Tejas Belagod <tejas.belagod@arm.com>,
	"gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: Re: [ARM] Fix CLZ_DEFINED_VALUE_AT_ZERO for vector modes
Date: Wed, 12 Nov 2014 12:43:00 -0000	[thread overview]
Message-ID: <CAKdteOZa6XjopwxM7TJuZmiM=aasvLycfe53qg3VJyTGhRyaUg@mail.gmail.com> (raw)
In-Reply-To: <DA41BE1DDCA941489001C7FBD7A8820E55553D71@szxema507-mbx.china.huawei.com>

On 12 November 2014 04:50, Yangfei (Felix) <felix.yang@huawei.com> wrote:
>> On Wed, Oct 22, 2014 at 10:49 PM, Michael Collison <michael.collison@linaro.org>
>> wrote:
>> >
>> > Patch that removes extraneous comment attached.
>> >
>> > The CLZ_DEFINED_VALUE_AT_ZERO macro is hard coded to return 32. For
>> > the vector intrinsic vclz this is incorrect and should return the
>> > value
>>
>> vclz_{s,u}8 ...
>>
>> > eight. The CTZ_DEFINED_VALUE_AT_ZERO has the same issue.
>>
>> The patch is ok . Sorry about the delay in reviewing this.
>>
>> Ramana
>>
>
>
> Hello,
>
>     Can anybody backport this patch for the 4.8/4.9 branch? They have the same issue too. Thanks.
>

If it does not reach FSF 4.8/4.9 branches, we'll probably backport it
to linaro-4.9 in our December release.

Christophe.

  reply	other threads:[~2014-11-12 12:36 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-09  7:05 Michael Collison
2014-10-09  7:11 ` Andrew Pinski
2014-10-09  7:26   ` Michael Collison
2014-10-09  7:55 ` Tejas Belagod
2014-10-09  7:56   ` Michael Collison
2014-10-22 21:53   ` Michael Collison
2014-10-31 18:44     ` Ramana Radhakrishnan
2014-11-12  3:51       ` Yangfei (Felix)
2014-11-12 12:43         ` Christophe Lyon [this message]
2014-11-12 13:11         ` Christophe Lyon
2014-11-12 13:48           ` Ramana Radhakrishnan
2014-11-13 13:54             ` Christophe Lyon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAKdteOZa6XjopwxM7TJuZmiM=aasvLycfe53qg3VJyTGhRyaUg@mail.gmail.com' \
    --to=christophe.lyon@linaro.org \
    --cc=felix.yang@huawei.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=michael.collison@linaro.org \
    --cc=ramrad01@arm.com \
    --cc=tejas.belagod@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).