On 12 November 2014 14:46, Ramana Radhakrishnan <ramana.radhakrish...@arm.com> wrote: > > > On 12/11/14 13:06, Christophe Lyon wrote: >> >> On 12 November 2014 04:50, Yangfei (Felix) <felix.y...@huawei.com> wrote: >>>> >>>> On Wed, Oct 22, 2014 at 10:49 PM, Michael Collison >>>> <michael.colli...@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. >>> >> >> Hi, >> >> I can take care of the backport, provided a maintainer such as Ramana >> approves it first. >> >> The original PR (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63365) >> is against 4.9.2, but the bug was present in 4.8 too. >> >> OK to backport it to 4.8 and 4.9? >> > > Ok by me unless an RM objects in the next 24 hours. >
Committed: - 4.9 branch: r217488. - 4.8 branch: r217490. > Ramana > > >> Thanks, >> >> Christophe. >> >