This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: cost model patch
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Xinliang David Li <davidxl at google dot com>
- Cc: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 1 Oct 2013 10:28:02 +0200
- Subject: Re: cost model patch
- Authentication-results: sourceware.org; auth=none
- References: <CAAkRFZ+_9YLa-EGF1xzrgW6QRh+So1bTo_njyjUPXj9S_t+jYw at mail dot gmail dot com> <CAFiYyc1_zii9XkwmzdJVd78zymgoQA=sydWcb2Y38m1u9tjKbA at mail dot gmail dot com> <524952A5 dot 3090900 at arm dot com> <CAAkRFZLdpJUqigyk+7b_XL9KrZDadVvKgu8KSgiaU7VPtDeg=A at mail dot gmail dot com>
On Mon, Sep 30, 2013 at 5:26 PM, Xinliang David Li <davidxl@google.com> wrote:
> Yes, that will do. Can you do it for me? I can't do testing easily
> on arm myself.
It also fails on x86_64 with -m32. I always test on x86_64 with
multilibs enabled:
make -k -j12 check RUNTESTFLAGS="--target_board=unix/\{,-m32\}"
Richard.
> 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
>>
>>