[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gcc-patches
Subject:    Re: cost model patch
From:       Xinliang David Li <davidxl () google ! com>
Date:       2013-09-30 15:26:42
Message-ID: CAAkRFZLdpJUqigyk+7b_XL9KrZDadVvKgu8KSgiaU7VPtDeg=A () mail ! gmail ! com
[Download RAW message or body]

Yes, that will do.  Can you do it for me? I can't  do testing easily
on arm myself.

thanks,

David




On Mon, Sep 30, 2013 at 3:29 AM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi Richard, David,
>
>> In principle yes.  Note that it changes the behavior of -O2
>> -ftree-vectorize
>> as -ftree-vectorize does not imply changing the default cost model.  I am
>> fine with that, but eventually this will have some testsuite fallout.
>
> Indeed I am observing a regression with this patch on arm-none-eabi in
> gcc.dg/tree-ssa/gen-vect-26.c.
>
> Seems that the cheap vectoriser model doesn't do unaligned stores (as
> expected I think?). Is adding -fvect-cost-model=dynamic to the test options
> the correct approach?
>
>
> Thanks,
> Kyrill
>
>
[prev in list] [next in list] [prev in thread] [next in thread]